[patch] Add discriminator support to gas .loc directive
Cary Coutant
ccoutant@google.com
Tue Apr 21 17:27:00 GMT 2009
>> Tested on i386. Is this OK for trunk?
>
> Almost, but not quite:
>
> * Please add a line to gas/NEWS mentioning the new feature and extend the
> description of the .loc directive in gas/doc/as.texinfo.
OK.
> * Why do you allow the user to set a value of 0 for the descriminator when
> only values greater than 0 are emitted ?
The default value of the discriminator "register" in the line table
state machine is 0, and is reset to that with each row that is
emitted, so we don't need to set the register to 0 when it's already
0.
> PS. When you submit the binutils/dwarf.c patch please could you also extend
> the gas/testsuite/gas/lns/lns-common-1.s test to cover the new sub-command.
Will do. I was looking for the right place to add a test for this.
-cary
More information about the Binutils
mailing list