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

2021-03-31 Thread Gedare Bloom
On Wed, Mar 31, 2021 at 10:42 AM Alex White wrote: > > On Wed, Mar 31, 2021 at 11:22 AM Gedare Bloom wrote: > > > > On Wed, Mar 31, 2021 at 10:05 AM Alex White wrote: > > > > > > The `rangeIndex` variable is 1 higher than the index at which the first > > > instruction address was found. This fix

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

2021-03-31 Thread Alex White
On Wed, Mar 31, 2021 at 11:22 AM Gedare Bloom wrote: > > On Wed, Mar 31, 2021 at 10:05 AM Alex White wrote: > > > > The `rangeIndex` variable is 1 higher than the index at which the first > > instruction address was found. This fixes the lookup after the loop to > > account for that fact. > > ---

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

2021-03-31 Thread Gedare Bloom
On Wed, Mar 31, 2021 at 10:05 AM Alex White wrote: > > The `rangeIndex` variable is 1 higher than the index at which the first > instruction address was found. This fixes the lookup after the loop to > account for that fact. > --- > tester/covoar/ObjdumpProcessor.cc | 2 +- > 1 file changed, 1 in

[PATCH] 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 lookup after the loop to account for that fact. --- tester/covoar/ObjdumpProcessor.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tester/covoar/ObjdumpProce