[PATCH] gas: correct ignoring of C-style number suffixes
Frager, Neal
neal.frager@amd.com
Tue Oct 31 10:05:02 GMT 2023
> First of all the respective original changes didn't deal with just 0 having such a suffix - this needs additional logic outside of integer_constant(). Further bogus suffixes having more than two L-s were accepted, while valid suffixes with U following the L(s) weren't.
> Finally respective tests were introduced for Sparc only.
> ---
> Instead of disabling the test for tic{,5}4x an alternative might be to use .dc.b. What I'm unclear about is whether that then really results in rubbish for those two targets (i.e. I'm uncertain whether they really shouldn't also override those directives).
Reviewed-by: Neal Frager <neal.frager@amd.com>
Thank you for developing this solution.
Best regards,
Neal Frager
AMD
More information about the Binutils
mailing list