Re: [PATCH] Restore input_location after recursive expand_call_inline

2021-01-05 Thread Bernd Edlinger
On 1/5/21 5:51 PM, Jeff Law wrote: > > > On 1/5/21 1:05 AM, Richard Biener wrote: >> On Tue, 5 Jan 2021, Bernd Edlinger wrote: >> >>> >>> On 1/4/21 10:23 PM, Jeff Law wrote: On 1/4/21 1:12 PM, Bernd Edlinger wrote: > Hi, > > I spotted a place where input_location is clobbere

Re: [PATCH] Restore input_location after recursive expand_call_inline

2021-01-05 Thread Jeff Law via Gcc-patches
On 1/5/21 1:05 AM, Richard Biener wrote: > On Tue, 5 Jan 2021, Bernd Edlinger wrote: > >> >> On 1/4/21 10:23 PM, Jeff Law wrote: >>> >>> On 1/4/21 1:12 PM, Bernd Edlinger wrote: Hi, I spotted a place where input_location is clobbered accidentally. That is in a recursive

Re: [PATCH] Restore input_location after recursive expand_call_inline

2021-01-05 Thread Richard Biener
On Tue, 5 Jan 2021, Bernd Edlinger wrote: > > > On 1/4/21 10:23 PM, Jeff Law wrote: > > > > > > On 1/4/21 1:12 PM, Bernd Edlinger wrote: > >> Hi, > >> > >> I spotted a place where input_location is clobbered accidentally. > >> > >> That is in a recursive call to expand_call_inline. The input_

Re: [PATCH] Restore input_location after recursive expand_call_inline

2021-01-04 Thread Bernd Edlinger
On 1/4/21 10:23 PM, Jeff Law wrote: > > > On 1/4/21 1:12 PM, Bernd Edlinger wrote: >> Hi, >> >> I spotted a place where input_location is clobbered accidentally. >> >> That is in a recursive call to expand_call_inline. The input_location >> is usually restored by goto egress in this function.

Re: [PATCH] Restore input_location after recursive expand_call_inline

2021-01-04 Thread Jeff Law via Gcc-patches
On 1/4/21 1:12 PM, Bernd Edlinger wrote: > Hi, > > I spotted a place where input_location is clobbered accidentally. > > That is in a recursive call to expand_call_inline. The input_location > is usually restored by goto egress in this function. > > Additionally the return value of the recursiv