Remove a test that was ported from the old llvm-gcc frontend test suite.
authorChandler Carruth <chandlerc@gmail.com>
Sat, 18 Oct 2014 06:36:18 +0000 (06:36 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Sat, 18 Oct 2014 06:36:18 +0000 (06:36 +0000)
commit71009cad9558af558549a0b8ff7fd1851833302d
treedab1c7d99d932b46dc1f8cec6b108171296266a6
parentee301f941c4e4b02eff76cb1b8f2f10b88327619
Remove a test that was ported from the old llvm-gcc frontend test suite.

This test is pretty awesome. It is claiming to test devirtualization.
However, the code in question is not in fact devirtualized by LLVM. If
you take the original C++ test case and run it through Clang at -O3 we
fail to devirtualize it completely. It also isn't a sufficiently focused
test case.

The *reason* we fail to devirtualize it isn't because of any missing
instcombine though. Instead, it is because we fail to emit an available
externally vtable and thus the vtable is just an external and completely
opaque. If I cause the vtable to be emitted, we successfully
devirtualize things.

Anyways, I'm just removing it because it is providing negative value at
this point: it isn't representative of the output of Clang really, LLVM
isn't doing the transform it claims to be testing, LLVM's failure to do
the transform isn't actually an LLVM bug at all and we shouldn't be
testing for it here, and finally the test is written in such a way that
it will trivially pass even when the point of the test is failing.

llvm-svn: 220137
llvm/test/Transforms/InstCombine/devirt.ll [deleted file]