[llvm-bugs] [Bug 31357] New: DAGCombiner::MatchBswapHword logic is wrong for OR trees

[via llvm-bugs]

https://llvm.org/bugs/show_bug.cgi?id=31357

            Bug ID: 31357
           Summary: DAGCombiner::MatchBswapHword logic is wrong for OR
                    trees
           Product: new-bugs
           Version: trunk
          Hardware: Other
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: new bugs
          Assignee: unassignedbugs at nondot.org
          Reporter: jlewis at knupath.com
                CC: llvm-bugs at lists.llvm.org
    Classification: Unclassified

Created attachment 17750
  --> https://llvm.org/bugs/attachment.cgi?id=17750&action=edit
Fix to DAGCombiner.cpp and two new unit test files

The existing code only looks at half of the tree when matching bswap + rol
patterns ending in an OR tree (as opposed to a cascade).

Proposed patch with test coverage is attached

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20161212/2dba62c2/attachment.html>