Patch: Remove non-mapped location support

Gabriel Dos Reis gdr@integrable-solutions.net
Mon Feb 25 19:49:00 GMT 2008


On Mon, Feb 25, 2008 at 12:56 PM, Manuel López-Ibáñez
<lopezibanez@gmail.com> wrote:
> On 25/02/2008, Chris Lattner <clattner@apple.com> wrote:
>
> >  It emits it on the marked line, which is *after the wrong plus*.  I
>  >  think GCC has a long way to go before it will have accurate location
>  >  info in general.  Relying on input_location at all seems very dangerous.
>
>  You are right. We should perhaps change the API of diagnostics to
>  force the caller to give a explicit location. After all, a lot of the
>  calls now incorporate "%H" at the beginning, which doesn't look great
>  either. If you want the input_location, just pass that.
>
>  I guess that implementing the clang ranges may be much more difficult
>  with our current diagnostics machinery (Gabriel? Tom? any idea about
>  this?). However, we should be able to do the caret.

I would like caret diagnostics for GCC -- it is long overdue.
We should contemplate changes that make it possible.
We should also be open to incremental work/improvements.

-- Gaby



More information about the Gcc-patches mailing list