[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