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

Kevin Athey via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 8 13:36:00 PDT 2021


kda added inline comments.


================
Comment at: compiler-rt/lib/asan/asan_fake_stack.cpp:219
 
+ALWAYS_INLINE uptr OnMallocAlways(uptr class_id, uptr size) {
+  FakeStack *fs = GetFakeStackFastAlways();
----------------
vitalybuka wrote:
> static ALWAYS_INLINE
> siblings as well.
Although, with ALWAYS_INLINE, I'm not sure 'static' makes any difference, as the code is already local to the call point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103304



More information about the llvm-commits mailing list