Re: [PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-29 Thread Dodji Seketeli
Jason Merrill writes: > On 04/25/2012 11:31 AM, Dodji Seketeli wrote: >> +#define EXPANSION_POINT_LOCATION_FILE(LOC) \ >> + ((expand_location_to_expansion_point (LOC)).file) >> +#define EXPANSION_POINT_LOCATION_LINE(LOC) \ >> + ((expand_location_to_expansion_point (LOC)).line)

Re: [PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-28 Thread Jason Merrill
On 04/25/2012 11:31 AM, Dodji Seketeli wrote: +#define EXPANSION_POINT_LOCATION_FILE(LOC) \ + ((expand_location_to_expansion_point (LOC)).file) +#define EXPANSION_POINT_LOCATION_LINE(LOC) \ + ((expand_location_to_expansion_point (LOC)).line) +#define EXPANSION_POINT_LOCA

Re: [PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-25 Thread Dodji Seketeli
I have re-based my tree on top of a recent tree that incorporates the changes for caret diagnostics and I had to update this patch accordingly. Here is its new version on trunk of today. It basically updates the new diagnostic_show_locus function to point to spelling locations. The patch does pa

Re: [PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-25 Thread Dodji Seketeli
> On 04/10/2012 03:49 PM, Dodji Seketeli wrote: >> Apparently, quite some places in the compiler (like the C/C++ >> preprocessor, the debug info machinery) expect expand_location to >> resolve to locations that are in the main source file, even if the >> token at stake comes from a macro that was d

Re: [PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-11 Thread Jason Merrill
On 04/10/2012 03:49 PM, Dodji Seketeli wrote: Apparently, quite some places in the compiler (like the C/C++ preprocessor, the debug info machinery) expect expand_location to resolve to locations that are in the main source file, even if the token at stake comes from a macro that was defined in a

[PATCH 05/11] Make expand_location resolve to locus in main source file

2012-04-10 Thread Dodji Seketeli
Apparently, quite some places in the compiler (like the C/C++ preprocessor, the debug info machinery) expect expand_location to resolve to locations that are in the main source file, even if the token at stake comes from a macro that was defined in a header somewhere. Turning on -ftrack-macro-expa