[llvm-commits] [llvm] r141203 - in /llvm/trunk: lib/Transforms/InstCombine/InstructionCombining.cpp test/Transforms/InstCombine/2010-11-01-lshr-mask.ll

[Jakob Stoklund Olesen]

On Oct 5, 2011, at 1:47 PM, Jim Grosbach wrote:

> 
> On Oct 5, 2011, at 1:41 PM, Jakob Stoklund Olesen wrote:
> 
>>>> After this commit, this test seems to be looping:
>>>> 
>>>> $ opt < test/Transforms/InstCombine/or.ll -instcombine -S
>>>> 
>>>> It is spending most of its time in SimplifyDemandedUseBits().
>>> 
>>> 
>>> Huh. That's odd. It's passing for me OK. Lemme fiddle and try to reproduce.
>> 
>> I just double-checked that reverting this patch made it terminate properly.
>> 
>> It's a Release+Asserts build.
> 
> Thanks. Something odd going on there. Reverted in 141209. I should be able to fix the --debug crash a bit more conservatively.

Thanks.

I was building with a clang from, say, a week ago.  I'll try self hosting with r141207 to see if it was a miscompilation.

/jakob