Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-12-02 Thread Pierre-Marie de Rodat
On 12/02/2015 02:57 PM, Jakub Jelinek wrote: Ok, thanks. Great, thank you! I’ve pushed the change. -- Pierre-Marie de Rodat

Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-12-02 Thread Jakub Jelinek
On Wed, Dec 02, 2015 at 02:46:28PM +0100, Pierre-Marie de Rodat wrote: > On 11/24/2015 06:10 PM, Jakub Jelinek wrote: > >The new pass is IMNSHO completely useless and undesirable, both for compile > >time (another whole IL traversal) reasons and for the unnecessary creation > >of memory allocations

Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-12-02 Thread Pierre-Marie de Rodat
On 11/24/2015 06:10 PM, Jakub Jelinek wrote: The new pass is IMNSHO completely useless and undesirable, both for compile time (another whole IL traversal) reasons and for the unnecessary creation of memory allocations. Understood. Thank you very much for explaining how you think it should be!

Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-11-30 Thread Pierre-Marie de Rodat
Hello Jakub, On 11/24/2015 06:10 PM, Jakub Jelinek wrote: The new pass is IMNSHO completely useless and undesirable, both for compile time (another whole IL traversal) reasons and for the unnecessary creation of memory allocations. […] Thank you for your detailed answer! This is just to say th

Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-11-24 Thread Jakub Jelinek
On Tue, Nov 24, 2015 at 05:31:11PM +0100, Pierre-Marie de Rodat wrote: > On 11/23/2015 10:11 PM, Jason Merrill wrote: > >Jakub, since DW_TAG_GNU_call_site is your feature, could you review this? > > As Jeff Law suggested in the “GCC 6 Status Report” thread, I’ve added > Alexandre Oliva to the disc

Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-11-24 Thread Pierre-Marie de Rodat
On 11/23/2015 10:11 PM, Jason Merrill wrote: Jakub, since DW_TAG_GNU_call_site is your feature, could you review this? As Jeff Law suggested in the “GCC 6 Status Report” thread, I’ve added Alexandre Oliva to the discussion to review the var-tracking part. Also, I’ve rebased+bootstrapped+regt

Re: [PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-11-23 Thread Jason Merrill
On 08/31/2015 03:28 AM, Pierre-Marie de Rodat wrote: On 07/20/2015 02:45 PM, Pierre-Marie de Rodat wrote: On PowerPC targets with -mlongcall, most subprogram calls are turned into indirect calls: the call target is read from a register even though it is compile-time known. This makes it difficul

[PATCH, PING*4] Track indirect calls for call site information in debug info.

2015-08-31 Thread Pierre-Marie de Rodat
On 07/20/2015 02:45 PM, Pierre-Marie de Rodat wrote: On PowerPC targets with -mlongcall, most subprogram calls are turned into indirect calls: the call target is read from a register even though it is compile-time known. This makes it difficult for machine code static analysis engines to recover