On Mon, Feb 6, 2017 at 7:00 AM, Prasad Ghangal <prasad.ghan...@gmail.com> wrote:
> On 4 January 2017 at 16:02, Richard Biener <richard.guent...@gmail.com> wrote:
>> On Wed, Dec 28, 2016 at 7:27 PM, Prasad Ghangal
>> <prasad.ghan...@gmail.com> wrote:
>>> Hi,
>>> The attached patch tries fix ICE due to undeclared variable(s) in the input.
>>> Successfully bootstrapped on x86_64-pc-linux-gnu, testing is in progress
>>
>> Ok.
>>
> Can you please commit the patch? I don't have access for that.

Can you share a testcase that broke?  I tried

int __GIMPLE foo(int a)
{
  if (t1 != 2)
    goto bb1;
  else
    goto bb2;

bb1:
  return t1;

bb2:
  return 1;
}

and it reports

t.c: In function ‘foo’:
t.c:3:7: error: ‘t1’ undeclared (first use in this function)
   if (t1 != 2)
       ^~
t.c:3:7: note: each undeclared identifier is reported only once for
each function it appears in
t.c:9:10: error: invalid conversion in return statement
   return t1;
          ^~
t.c:1:14: note: declared here
 int __GIMPLE foo(int a)
              ^~~

and thus doesn't ICE.

Maybe one of my patches in this area made yours redundant (it doesn't
apply cleanly anymore as well).

Thanks,
Richard.

> Thanks,
> Prasad
>
>> Richard.
>>
>>>
>>> Thanks,
>>> Prasad

Reply via email to