[PATCH] D76886: [InlineFunction] Disable emission of alignment assumptions by default
    Roman Lebedev via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Mar 27 07:37:08 PDT 2020
    
    
  
lebedev.ri added inline comments.
================
Comment at: test/Transforms/PhaseOrdering/inlining-alignment-assumptions.ll:3
+; RUN: opt -S -O2 -preserve-alignment-assumptions-during-inlining=0 < %s | FileCheck %s --check-prefixes=CHECK,ASSUMPTIONS-OFF
+; RUN: opt -S -O2 -preserve-alignment-assumptions-during-inlining=1 < %s | FileCheck %s --check-prefixes=CHECK,ASSUMPTIONS-ON
+
----------------
Err, one thing i forgot to add: i mainly wanted to see the test that exercises
the *default* value of `preserve-alignment-assumptions-during-inlining`.
So i think this needs one more line like 
```
; RUN: opt -S -O2 -preserve-alignment-assumptions-during-inlining=0 < %s | FileCheck %s --check-prefixes=CHECK,ASSUMPTIONS-OFF,FALLBACK-0
; RUN: opt -S -O2 -preserve-alignment-assumptions-during-inlining=1 < %s | FileCheck %s --check-prefixes=CHECK,ASSUMPTIONS-ON,FALLBACK-1
; RUN: opt -S -O2 < %s | FileCheck %s --check-prefixes=CHECK,ASSUMPTIONS-OFF,FALLBACK-DEFAULT
```
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76886/new/
https://reviews.llvm.org/D76886
    
    
More information about the llvm-commits
mailing list