[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 planned changes to this revision.
shawnl marked an inline comment as done.
shawnl added a comment.
I'll get rid of the leftovers.
================
Comment at: test/Transforms/SimplifyCFG/switch-on-const-select.ll:158
; CHECK: attributes [[$NUW]] = { nounwind }
; CHECK: attributes #1 = { cold noreturn nounwind }
----------------
nikic wrote:
> Does this test actually pass after being regenerated? I'd expect an error due to the undefined variable here. This probably needs to be replaced with just `#0` if you're using it above.
You are referring to the other file, but yes it fails, but no, this is not a bug in LLVM, but rather a bug in the test.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61150/new/
https://reviews.llvm.org/D61150
More information about the llvm-commits
mailing list