Re: [PATCH 4/7] Support -fdebug-cpp option

2011-09-17 Thread Jason Merrill
OK. Jason

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-09-16 Thread Dodji Seketeli
Jason Merrill writes: > On 08/24/2011 10:06 AM, Tom Tromey wrote: >> Dodji> Would it be acceptable to just change the output of -fdirective to >> fit? >> Dodji> Or are we bound to not breaking existing consumers? >> >> I think changing it would be fine. > > I agree. I have added that to my TODO

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-09-12 Thread Jason Merrill
On 08/24/2011 10:06 AM, Tom Tromey wrote: Dodji> Would it be acceptable to just change the output of -fdirective to fit? Dodji> Or are we bound to not breaking existing consumers? I think changing it would be fine. I agree. On 07/16/2011 10:37 AM, Dodji Seketeli wrote: } + +void +linemap_du

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-08-24 Thread Tom Tromey
Tom> That exists -- -fdirectives-only. Jakub> It isn't exactly what would be needed, as e.g. \\\n are removed from Jakub> from #defines and thus they get different location of the tokens. Dodji> Would it be acceptable to just change the output of -fdirective to fit? Dodji> Or are we bound to not

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-08-23 Thread Dodji Seketeli
Alexandre Oliva writes: > On Jul 16, 2011, Dodji Seketeli wrote: > >> This patch adds -fdebug-cpp option. When used with -E this dumps the >> relevant macro map before every single token. This clutters the output >> a lot but has proved to be invaluable in tracking some bugs during the >> develo

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-08-22 Thread Jakub Jelinek
On Mon, Aug 22, 2011 at 08:16:45AM -0600, Tom Tromey wrote: > > "Jakub" == Jakub Jelinek writes: > > Jakub> For ccache and friends I think it would be better to have a > Jakub> preprocessing mode that would output all lines as is (i.e. no > Jakub> macro replacement), except for processing #in

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-08-22 Thread Tom Tromey
> "Jakub" == Jakub Jelinek writes: Jakub> For ccache and friends I think it would be better to have a Jakub> preprocessing mode that would output all lines as is (i.e. no Jakub> macro replacement), except for processing #include/#include_next Jakub> directives. That exists -- -fdirectives-on

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-08-21 Thread Jakub Jelinek
On Sun, Aug 21, 2011 at 04:51:54AM -0300, Alexandre Oliva wrote: > On Jul 16, 2011, Dodji Seketeli wrote: > > > This patch adds -fdebug-cpp option. When used with -E this dumps the > > relevant macro map before every single token. This clutters the output > > a lot but has proved to be invaluable

Re: [PATCH 4/7] Support -fdebug-cpp option

2011-08-21 Thread Alexandre Oliva
On Jul 16, 2011, Dodji Seketeli wrote: > This patch adds -fdebug-cpp option. When used with -E this dumps the > relevant macro map before every single token. This clutters the output > a lot but has proved to be invaluable in tracking some bugs during the > development of the virtual location sup