[llvm] r257940 - [SelectionDAG] CSE nodes with differing SDNodeFlags

[James Molloy via llvm-commits]

Hi Hans,

Yes please!

James
> On 20 Jan 2016, at 18:05, Hans Wennborg <hans at chromium.org> wrote:
>
> James, is this something that needs to be merged to 3.8?
>
> On Fri, Jan 15, 2016 at 1:56 PM, Dan Gohman via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> Author: djg
>> Date: Fri Jan 15 15:56:40 2016
>> New Revision: 257940
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=257940&view=rev
>> Log:
>> [SelectionDAG] CSE nodes with differing SDNodeFlags
>>
>> In the optimizer (GVN etc.) when eliminating redundant nodes with different
>> flags, the flags are ignored for the purposes of testing for congruence, and
>> then intersected for the purposes of producing a result that supports the union
>> of all the uses. This commit makes SelectionDAG's CSE do the same thing,
>> allowing it to CSE nodes in more cases. This fixes PR26063.
>>
>> Differential Revision: http://reviews.llvm.org/D15957
>

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.