vitalybuka added inline comments.
================ Comment at: llvm/lib/Transforms/Instrumentation/HWAddressSanitizer.cpp:406 + // architecture that doesn't allow stack tagging we will still load the + // analysis. + // This is so we don't need to plumb TargetTriple all the way to here. ---------------- I like this even less than "plumbing", if you need shouldUseStackSafetyAnalysis please just pass Triple as in earlier versions. Why do you need to avoid loading analysis? Revert does not explain what is broken. 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