This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PR 15236 pedantic switch modifies treatment of non-ISO compliant enumerations


>>>>> "Manuel" == Manuel LÃpez-IbÃÃez <lopezibanez@gmail.com> writes:

Manuel> Still, without enabling -fshow-column by default it seems hard to
Manuel> notice when we are getting the wrong location if it is in the same
Manuel> line.

Tom> Yeah.

Manuel> So why don't we do it for GCC 4.4? Anything that parses the output of
Manuel> GCC needs to handle the output of cpp and cpp already prints column
Manuel> numbers. And there is always -fno-show-column.

Just to be clear -- I couldn't approve or reject a change here.
I guess you could submit a patch and see what happens :)

FWIW, my opinion is that we should fix the locations first, and only
then turn on column output.  My reason is just that I have tried this
already, and I know that our location handling is pretty bad.  And, it
is a pretty big effort to clean this up.  I think this is well worth
doing, but it would take a concerted effort to finish for 4.4.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]