On Sun, 15 Jun 2014, Steven Bosscher wrote: > On Sun, Jun 15, 2014 at 1:27 PM, Hans-Peter Nilsson wrote: > > /tmp/hpautotest-gcc0/gcc/gcc/auto-inc-dec.c: In function 'void > > merge_in_block(int, basic_block_def*)': > > /tmp/hpautotest-gcc0/gcc/gcc/auto-inc-dec.c:1442: error: 'uid' > > was not declared in this scope > > make[2]: *** [auto-inc-dec.o] Error 1 > > > > brgds, H-P > > > Bah, this is why I just *hate* all the gcc code that's only compiled > if certain #defines are set...
I couldn't agree more. Might not have been obvious when writing the mosly-mechanical patch, still the auto-inc-dec.c name should have been a red flag that a representative target should have been tested (i.e. not x86_64 and i686). > > Can you please try: > > [...] Thanks. Looks pretty obvious. I was heading for the door with just enough time to report the issue, so I didn't actually look at the code before. I'll commit this on your behalf once build has passed the point of failure. brgds, H-P
