Revert patch series introducing the DAG combine to match a load-by-bytes
authorChandler Carruth <chandlerc@gmail.com>
Fri, 16 Dec 2016 04:05:22 +0000 (04:05 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Fri, 16 Dec 2016 04:05:22 +0000 (04:05 +0000)
commit4154062b6902862e987fb5b62293e0ff5915cbe9
tree08ebe21e281be7ac6b0e73ff43da0c80f4b81d31
parentc1ded7dcef3fcb94fe95fd0105b220b54c515158
Revert patch series introducing the DAG combine to match a load-by-bytes
idiom.

r289538: Match load by bytes idiom and fold it into a single load
r289540: Fix a buildbot failure introduced by r289538
r289545: Use more detailed assertion messages in the code ...
r289646: Add a couple of assertions to the load combine code ...

This DAG combine has a bad crash in it that is quite hard to trigger
sadly -- it relies on sneaking code with UB through the SDAG build and
into this particular combine. I've responded to the original commit with
a test case that reproduces it.

However, the code also has other problems that will require substantial
changes to address and so I'm going ahead and reverting it for now. This
should unblock us and perhaps others that are hitting the crash in the
wild and will let a fresh patch with updated approach come in cleanly
afterward.

Sorry for any trouble or disruption!

llvm-svn: 289916
llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
llvm/test/CodeGen/ARM/load-combine-big-endian.ll [deleted file]
llvm/test/CodeGen/ARM/load-combine.ll [deleted file]
llvm/test/CodeGen/X86/load-combine.ll [deleted file]