vitalybuka wrote: > We can set `-mllvm -stack-safety-run` to force `getInfo`, but it seems that > the current lazy `getInfo` works as intended once this patch is applied.
No, we don't need any new flag. We need to trigger getInfo before any module modification, e.g. at getResult<StackSafetyGlobalAnalysis> if some function without alloca replaces mem intrinsic, results of calculation of StackSafetyGlobalAnalysis after that will be incorrect (still it's just missed optimization, unnecessary instrumentation). Any way, no need to do so in this patch. https://github.com/llvm/llvm-project/pull/77221 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
