[PATCH] D112098: [ASan] Added stack safety support in address sanitizer.

[Florian Mayer via Phabricator via cfe-commits]

fmayer added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp:812
   bool runOnFunction(Function &F) override {
+    if (ClUseStackSafety) {
+      report_fatal_error("Stack safety analysis is not supported "
----------------
kstoimenov wrote:
> vitalybuka wrote:
> > functionality of the pass should be the same, so I don't see why this is the fatal error. so just ignore it?
> > 
> > btw, why don't you want to support it?
> I would like to support it, but I am not sure how to get an instance of StackSafetyGlobalAnalysis, because in the legacy pass manger I don't have access to AnalysisManager<Function> &AM. If you know how to make this work, please let me know. 
> 
> As far as the report_fatal_error my logic is that if someone sets ClUseStackSafety flag they expect it to work and if it silently doesn't it is a bad user experience. 
Maybe it would help to look at the HWASan instrumentation pass, which uses the stack safety analysis for both old and new pass managers.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112098/new/

https://reviews.llvm.org/D112098