[PATCH v7 0/2] Fix NOP recognition

Alex White alex.white at oarcorp.com
Fri Mar 26 16:40:10 UTC 2021


v7:
- Add comment clarifying `using` declaration in CoverageMapNotFoundError class

v6:
- Consolidate loop conditionals in Coverage::finalizeSymbol()

v5:
- Fix missing std::dec at the end of error message printing in
  Coverage::finalizeSymbol()

v4:
- Add specialized CoverageMapNotFoundError exception class to ExecutableInfo
- Catch ExecutableInfo::CoverageMapNotFoundError in Coverage::finalizeSymbol()

v3:
- Fix double increment of rangeIndex in Coverage::finalizeSymbol()

v2:
- Fix leftover debugging code in Coverage::finalizeSymbol()

This patch set fixes a couple cases where NOP instructions were showing up as unexecuted in coverage reports.

Alex White (2):
  covoar: Fix NOP execution marking
  covoar/Target_i386: Add NOP patterns

 tester/covoar/CoverageMapBase.cc  |  10 +++
 tester/covoar/CoverageMapBase.h   |   4 ++
 tester/covoar/DesiredSymbols.cc   |  38 +++++++++--
 tester/covoar/DesiredSymbols.h    |  11 +++-
 tester/covoar/ExecutableInfo.cc   |   2 +-
 tester/covoar/ExecutableInfo.h    |   5 ++
 tester/covoar/ObjdumpProcessor.cc | 105 ++++++++++++++++++++++--------
 tester/covoar/Target_i386.cc      |   9 +++
 8 files changed, 148 insertions(+), 36 deletions(-)

-- 
2.27.0



More information about the devel mailing list