[PATCH] D82124: [StackSafety] Add "Must Live" logic
    Evgenii Stepanov via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Jun 18 14:15:04 PDT 2020
    
    
  
eugenis added inline comments.
================
Comment at: llvm/test/Analysis/StackSafetyAnalysis/lifetime.ll:2
+; RUN: opt -analyze -stack-lifetime -stack-lifetime-must=0 -S < %s -o - | FileCheck %s --check-prefixes=CHECK,MAY
+; RUN: opt -analyze -stack-lifetime -stack-lifetime-must=1 -S < %s -o - | FileCheck %s --check-prefixes=CHECK,MUST
 
----------------
It would be nicer to make it a parametrized pass, something like this:
  -passes=stack-lifetime<must-live> or stack-lifetime<may-live>
And turn the bool argument into an enum with named options.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82124/new/
https://reviews.llvm.org/D82124
    
    
More information about the llvm-commits
mailing list