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: [C++ Patch] PR 44516


On 05/22/2012 08:03 PM, Mike Stump wrote:
On May 22, 2012, at 8:42 AM, Jason Merrill wrote:
On 05/22/2012 08:22 AM, Paolo Carlini wrote:
Ok, thanks. Today I wanted to concretely test and post a patch but I
noticed this other objc bit in objc-next-runtime-abi-01.c which I don't
know how to handle due to the OPT_Wall gate:

warning_at (UNKNOWN_LOCATION, OPT_Wall,
"%<-fobjc-sjlj-exceptions%>  is the only supported exceptions "
"system for %<-fnext-runtime%>  with %<-fobjc-abi-version%>  <  2");

If the objc maintainers could help with those uses of warning_at (0 in
the objc tree, I have a patch ready for the asserts proper.
What if we just changed UNKNOWN_LOCATION to BUILTINS_LOCATION for these diagnostics?
I thought about that, but only after wanting to use COMMANDLINE_LOCATION, looking around and not finding it.... If everyone else is using BUILTINS_LOCATION, that's fine, but just seems wrong. UNKNOWN almost seems like a better match at first blush.
BUILTINS_LOCATION sounds a bit weird to me too. I guess, if we do that we need clear comments. And in any case, the issue, as stupid as it seems in principle, turns out to be quite tricky, because the driver really expects UNKNOWN_LOCATION in quite a few places, there are even asserts. Just doing search&replace for the UNKNOWN_LOCATIONs passed by warning_at and error_at doesn't work.

Paolo.


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