[PATCH] D61150: [SimplifyCFG] NFC, update Switch tests to HEAD so I can see if my changes change anything

Shawn Landden via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 13 16:17:22 PDT 2019


shawnl marked an inline comment as done.
shawnl added a comment.

comment didn't get included.



================
Comment at: test/Transforms/SimplifyCFG/switch-dead-default.ll:246
 ; CHECK-LABEL: @test8(
-; CHECK: switch i8
+; CHECK-NEXT:    unreachable
+;
----------------
nikic wrote:
> This seems odd. Are you sure these are the results without any other patches?
Yes, and it is correct. I opened a bug and then was corrected that this is correct.  Here is the patch removing this buggy test. https://reviews.llvm.org/D60859


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61150/new/

https://reviews.llvm.org/D61150





More information about the llvm-commits mailing list