Re: [PATCH v2] covoar: Fix off-by-one in Coverage::finalizeSymbol()

2021-03-31 Thread Gedare Bloom
ok On Wed, Mar 31, 2021 at 11:22 AM Alex White wrote: > > The `rangeIndex` variable is 1 higher than the index at which the first > instruction address was found. This fixes the loop to set `rangeIndex` > correctly. > --- > tester/covoar/ObjdumpProcessor.cc | 9 - > 1 file changed, 4 ins

[PATCH v2] covoar: Fix off-by-one in Coverage::finalizeSymbol()

2021-03-31 Thread Alex White
The `rangeIndex` variable is 1 higher than the index at which the first instruction address was found. This fixes the loop to set `rangeIndex` correctly. --- tester/covoar/ObjdumpProcessor.cc | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/tester/covoar/ObjdumpProcesso