Re: demand_empty_rest_of_line and ignore_rest_of_line

[Zack Weinberg <zack at codesourcery dot com>]

Hans-Peter Nilsson <hp@bitrange.com> writes:

> That conclusion is incorrect.  #NO_APP for cris-* is output
> through default_file_start.  It seems there are multiple blessed
> ways to get that #NO_APP out.  Hmm, I thought that kind of
> multiplicity was something Zack W disliked but apparently not. ;-)

I *tried*.  Y'all keep moving the goalposts.  ;-)

>> Maybe the #APP/#NO_APP switching should be removed and input scrubbing
>> enabled all the time, given how few targets actually disable it.
>
> See binutils archives from last time this came up (search for
> "no_app").  It saves as much as 1% off the compile time.  I
> think *more* targets should use it, but most would need to tweak
> their md:s to avoid redundant spaces.

Frankly, (as someone who has to stare at GCC's assembly output all the
damn time), I do not think 1% speedup is worth the *severe*
degradation in readability that this would impose.  I would much
rather put effort into making GAS's parser be faster in
input-scrubbing mode.

zw