[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