summaryrefslogtreecommitdiffstats
path: root/lldb/packages/Python/lldbsuite/test/python_api
diff options
context:
space:
mode:
authorSanjay Patel <spatel@rotateright.com>2017-02-27 21:30:54 +0000
committerSanjay Patel <spatel@rotateright.com>2017-02-27 21:30:54 +0000
commitae7873fe551b4bd0cfc24f86f9d82fb688e96297 (patch)
tree2e2b44bc002d1c0ee7053e6784b4b2a0eefc88c7 /lldb/packages/Python/lldbsuite/test/python_api
parentb7acfc0139bba6f5fdc3f4c409411f03dce543ab (diff)
downloadbcm5719-llvm-ae7873fe551b4bd0cfc24f86f9d82fb688e96297.tar.gz
bcm5719-llvm-ae7873fe551b4bd0cfc24f86f9d82fb688e96297.zip
[ARM] don't transform an add(ext Cond), C to select unless there's a setcc of the condition
The transform in question claims to be doing: // fold (add (select cc, 0, c), x) -> (select cc, x, (add, x, c)) ...starting in PerformADDCombineWithOperands(), but it wasn't actually checking for a setcc node for the sext/zext patterns. This is exactly the opposite of a transform I'd like to add to DAGCombiner's foldSelectOfConstants(), so I was seeing infinite loops with my draft of a patch applied. The changes in select_const.ll look positive (less instructions). The change in arm-and-tst-peephole.ll is unrelated. We're changing the input IR in that test to preserve the intent of the test, but that's not affected by this code change. Differential Revision: https://reviews.llvm.org/D30355 llvm-svn: 296389
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud