[llvm] r205484 - Add test case for [Constant Hoisting] Erase dead cast instructions (r204538).

[Juergen Ributzka]

Unfortunately there is nothing observable in the IR (like an extra instruction) I could have tested for.

In this particular case I removed a no longer needed instruction, because it had an empty use list.
The instruction was only removed, but not deleted. Later on when the destructor is called it hit an
assertion.

I will change the test to limit is to assert builds, as suggested by Eric, and check for the expected
output.

-Juergen

On Apr 3, 2014, at 8:25 AM, David Blaikie <dblaikie at gmail.com> wrote:

> For that matter - tests that simply test that we don't crash always
> seem rather suspect. What is the behavior we expect that we weren't
> getting due to the crash being in the way? We should really test for
> that.
> 
> - David

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140403/0cf28c46/attachment.html>