vitalybuka added inline comments.
================ Comment at: llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp:410 + Dummy.setArch(Triple::aarch64); + if (shouldUseStackSafetyAnalysis(Dummy, DisableOptimization)) + AU.addRequired<StackSafetyGlobalInfoWrapperPass>(); ---------------- Why not just: ``` if (!DisableOptimization || ClUseStackSafety) AU.addRequired<StackSafetyGlobalInfoWrapperPass>(); ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105703/new/ https://reviews.llvm.org/D105703 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
