A bad change to gas
Alan Modra
alan@linuxcare.com.au
Sat Jun 3 21:18:00 GMT 2000
On Sat, 3 Jun 2000, H . J . Lu wrote:
> > The proper fix is to go through all the gas/config/tc files and explicitly
> > add the `;' to line_separator_chars. I think it's worth doing things
>
> I tried it first. But I am not so convinced that it is a right way
> to go. It may break some platforms. If `;' is not a line separator on
> a platform by default, it is ok to treat it as a line separator unless
> it is for something else. In case of hppa, `;' is a comment char.
> It is set by comment_chars. It will override is_end_of_line.
Not always. Some (probably irrelevant) examples: "as -f ..." or #NO_APP
> > correctly as we get to remove a target dependent #ifdef from the
> > supposedly target independent files.
> >
>
> Could you please provide a testcase before you make any changes? I'd
> like to keep the way it is unless there is a testcase to show the
> current code is wrong.
H.J., some changes to code are to fix bugs, and a testcase is
appropriate. Other changes are for correctness of a more theoretical
sort, for elegance, and for maintainability. Any further tidying up
of is_end_of_line falls into the latter category.
Regards, Alan Modra
--
Linuxcare. Support for the Revolution.
More information about the Binutils
mailing list