[PATCH] D103304: Update and improve compiler-rt tests for -mllvm -asan_use_after_return=(never|[runtime]|always).

[Matt Morehouse via Phabricator via llvm-commits]

morehouse added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/AddressSanitizer.cpp:2654
+          M, IntTy, /*isConstant=*/true, GlobalValue::WeakODRLinkage,
+          ConstantInt::get(IntTy, 1), "__asan_detect_use_after_return_always");
+    });
----------------
kda wrote:
> morehouse wrote:
> > The constant value 1 is also inconsistent with the comment in the runtime (should be 2).
> I could change it if you felt it made the code clearer.  __asan_detect_use_after_return_always is functionally a boolean.  It is only created if ClUseAfterReturn is equal to Always (2).
Maybe just change the comment in asan_rtl.cpp.  Seems simplest.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103304