[PATCH] D61145: [NFC][Sanitizer] Change "return type" of INTERCEPT_FUNCTION to void

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: compiler-rt/lib/asan/asan_interceptors.h:127
+    bool same = (& (name) == & WRAP(name));                              \
+    if ((!same || !REAL(name)))                                          \
       VReport(1, "AddressSanitizer: failed to intercept '" #name "'\n"); \
----------------
yln wrote:
> You said this protects against "double interception"?
> If this works as expected, then should we push it up into INTERCEPT_FUNCTION?
> Why do we only protect against "double interception" in ASan and MSan, but not the other sanitizers?
> 
> My best guess is that this has been obsolete for a while.
Yes, I would like to put this into INTERCEPT_FUNCTION
however maybe VReport is not available in all users 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61145