[PATCH] D59140: [CGP] Fix UB when removing trivial PHINode
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 8 11:51:27 PST 2019
efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.
LGTM with one nit.
================
Comment at: test/CodeGen/AArch64/cgp-trivial-phi-node.ll:3
+; bound to trivial PHI node.
+; REQUIRES: asserts
+; RUN: opt -mtriple=aarch64-linux-gnu -codegenprepare -S -o - | FileCheck %s
----------------
Don't need REQUIRES: asserts; presumably the test should still behave the same way even without assertions. :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59140/new/
https://reviews.llvm.org/D59140
More information about the llvm-commits
mailing list