summaryrefslogtreecommitdiffstats
path: root/llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
diff options
context:
space:
mode:
authorChandler Carruth <chandlerc@gmail.com>2017-09-15 22:23:27 +0000
committerChandler Carruth <chandlerc@gmail.com>2017-09-15 22:23:27 +0000
commitbeb22b543718734ae62fc7325aa73d6708b2a42f (patch)
treeb3b797208c109f2fd79191685982334794d8393b /llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp
parent02a6fc5e8f4821844f7e30d8902943b7f8809a58 (diff)
downloadbcm5719-llvm-beb22b543718734ae62fc7325aa73d6708b2a42f.tar.gz
bcm5719-llvm-beb22b543718734ae62fc7325aa73d6708b2a42f.zip
[SLP] Revert r312791 and other necessary commits, except for TTI and
CostModel. The original patch added support for horizontal min/max reductions to the SLP vectorizer. This patch causes LLVM to miscompile fairly simple signed min reductions. I have attached a test progrom to http://llvm.org/PR34635 that shows the behavior change after this patch. We found this in a test for the open source Eigen library, but also in other code. Unfortunately, the revert is moderately challenging. It required reverting: r313042: [SLP] Test with multiple uses of conditional op and wrong parent. r312853: [SLP] Fix buildbots, NFC. r312793: [SLP] Fix the warning about paths not returning the value, NFC. r312791: [SLP] Support for horizontal min/max reduction. And even then, I had to completely skip reverting the changes to TTI and CostModel because r312832 rewrote so much of this code. Plus, the cost modeling changes aren implicated in the miscompile, so they should be fine and will just not be used until this gets re-introduced. llvm-svn: 313409
Diffstat (limited to 'llvm/tools/llvm-dwarfdump/llvm-dwarfdump.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud