[llvm] r237539 - Reapply r237520 with another fix for infinite looping

[James Molloy]

Thanks Hans!

I did have a sneaky suspicion it might.

Cheers,

James



> On 19 May 2015, at 19:11, Hans Wennborg <hans at chromium.org> wrote:
>
>> On Sun, May 17, 2015 at 1:27 AM, James Molloy <james.molloy at arm.com> wrote:
>> Author: jamesm
>> Date: Sun May 17 03:27:27 2015
>> New Revision: 237539
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=237539&view=rev
>> Log:
>> Reapply r237520 with another fix for infinite looping
>>
>> SimplifyDemandedBits was "simplifying" a constant by removing just sign bits.
>> This caused a canonicalization race between different parts of instcombine.
>>
>> Fix and regression test added - third time lucky?
>
> Sorry, but it seems this broke some tests in Chromium. I reduced it to
> PR23583. Reverted in r237739.
>
> - Hans
>

-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.

ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2557590
ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No:  2548782