[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