[PATCH] D114558: [ASan] Shared optimized callbacks implementation.

Fri Dec 3 12:15:52 PST 2021

vitalybuka added a comment.

In D114558#3167454 <https://reviews.llvm.org/D114558#3167454>, @vitalybuka wrote:

> I would recommend to separate llvm into a separate patch, so you can land compiler-rt separately. That part is likely NFC on it's own, but you can wait some time to see if it was not reverted because of breaking. And then you can land/revert llvm part as many times you want

This one is unrelieved. I don't insist. Just checking if you seen this.

================
Comment at: compiler-rt/lib/asan/asan_interface.inc:184
+
+// clang-format off
+#if defined(__x86_64__)
----------------
kstoimenov wrote:
> kstoimenov wrote:
> > vitalybuka wrote:
> > > vitalybuka wrote:
> > > > why?
> > > > I applied and clang-format does a reasonable job
> > > btw 
> > > check-asan fails like that 
> > > ```
> > > === NOTE === If you see a mismatch below, please update sanitizer_interface.inc files.
> > > 7,19c7,136
> > > < __asan_check_op_add_s_R12
> > > < __asan_check_op_add_s_R13
> > > < __asan_check_op_add_s_R14
> > > < __asan_check_op_add_s_R15
> > > < __asan_check_op_add_s_R8
> > > < __asan_check_op_add_s_R9
> > > < __asan_check_op_add_s_RAX
> > > < __asan_check_op_add_s_RBP
> > > < __asan_check_op_add_s_RBX
> > > < __asan_check_op_add_s_RCX
> > > < __asan_check_op_add_s_RDI
> > > < __asan_check_op_add_s_RDX
> > > < __asan_check_op_add_s_RSI
> > > ---
> > > > __asan_check_load_add_16_R12
> > > > __asan_check_load_add_16_R13
> > > > __asan_check_load_add_16_R14
> > > > __asan_check_load_add_16_R15
> > > > __asan_check_load_add_16_R8
> > > > __asan_check_load_add_16_R9
> > > > __asan_check_load_add_16_RAX
> > > > __asan_check_load_add_16_RBP
> > > > __asan_check_load_add_16_RBX
> > > > __asan_check_load_add_16_RCX
> > > > __asan_check_load_add_16_RDI
> > > > __asan_check_load_add_16_RDX
> > > > __asan_check_load_add_16_RSI
> > > > __asan_check_load_add_1_R12
> > > > __asan_check_load_add_1_R13
> > > > __asan_check_load_add_1_R14
> > > > __asan_check_load_add_1_R15
> > > > __asan_check_load_add_1_R8
> > > > __asan_check_load_add_1_R9
> > > > __asan_check_load_add_1_RAX
> > > > __asan_check_load_add_1_RBP
> > > > __asan_check_load_add_1_RBX
> > > > __asan_check_load_add_1_RCX
> > > > __asan_check_load_add_1_RDI
> > > > __asan_check_load_add_1_RDX
> > > > __asan_check_load_add_1_RSI
> > > > __asan_check_load_add_2_R12
> > > > __asan_check_load_add_2_R13
> > > > __asan_check_load_add_2_R14
> > > > __asan_check_load_add_2_R15
> > > > __asan_check_load_add_2_R8
> > > > __asan_check_load_add_2_R9
> > > > __asan_check_load_add_2_RAX
> > > > __asan_check_load_add_2_RBP
> > > > __asan_check_load_add_2_RBX
> > > > __asan_check_load_add_2_RCX
> > > > __asan_check_load_add_2_RDI
> > > > __asan_check_load_add_2_RDX
> > > > __asan_check_load_add_2_RSI
> > > > __asan_check_load_add_4_R12
> > > > __asan_check_load_add_4_R13
> > > > __asan_check_load_add_4_R14
> > > > __asan_check_load_add_4_R15
> > > > __asan_check_load_add_4_R8
> > > > __asan_check_load_add_4_R9
> > > > __asan_check_load_add_4_RAX
> > > > __asan_check_load_add_4_RBP
> > > > __asan_check_load_add_4_RBX
> > > > __asan_check_load_add_4_RCX
> > > > __asan_check_load_add_4_RDI
> > > > __asan_check_load_add_4_RDX
> > > > __asan_check_load_add_4_RSI
> > > > __asan_check_load_add_8_R12
> > > > __asan_check_load_add_8_R13
> > > > __asan_check_load_add_8_R14
> > > > __asan_check_load_add_8_R15
> > > > __asan_check_load_add_8_R8
> > > > __asan_check_load_add_8_R9
> > > > __asan_check_load_add_8_RAX
> > > > __asan_check_load_add_8_RBP
> > > > __asan_check_load_add_8_RBX
> > > > __asan_check_load_add_8_RCX
> > > > __asan_check_load_add_8_RDI
> > > > __asan_check_load_add_8_RDX
> > > > __asan_check_load_add_8_RSI
> > > > __asan_check_store_add_16_R12
> > > > __asan_check_store_add_16_R13
> > > > __asan_check_store_add_16_R14
> > > > __asan_check_store_add_16_R15
> > > > __asan_check_store_add_16_R8
> > > > __asan_check_store_add_16_R9
> > > > __asan_check_store_add_16_RAX
> > > > __asan_check_store_add_16_RBP
> > > > __asan_check_store_add_16_RBX
> > > > __asan_check_store_add_16_RCX
> > > > __asan_check_store_add_16_RDI
> > > > __asan_check_store_add_16_RDX
> > > > __asan_check_store_add_16_RSI
> > > > __asan_check_store_add_1_R12
> > > > __asan_check_store_add_1_R13
> > > > __asan_check_store_add_1_R14
> > > > __asan_check_store_add_1_R15
> > > > __asan_check_store_add_1_R8
> > > > __asan_check_store_add_1_R9
> > > > __asan_check_store_add_1_RAX
> > > > __asan_check_store_add_1_RBP
> > > > __asan_check_store_add_1_RBX
> > > > __asan_check_store_add_1_RCX
> > > > __asan_check_store_add_1_RDI
> > > > __asan_check_store_add_1_RDX
> > > > __asan_check_store_add_1_RSI
> > > > __asan_check_store_add_2_R12
> > > > __asan_check_store_add_2_R13
> > > > __asan_check_store_add_2_R14
> > > > __asan_check_store_add_2_R15
> > > > __asan_check_store_add_2_R8
> > > > __asan_check_store_add_2_R9
> > > > __asan_check_store_add_2_RAX
> > > > __asan_check_store_add_2_RBP
> > > > __asan_check_store_add_2_RBX
> > > > __asan_check_store_add_2_RCX
> > > > __asan_check_store_add_2_RDI
> > > > __asan_check_store_add_2_RDX
> > > > __asan_check_store_add_2_RSI
> > > > __asan_check_store_add_4_R12
> > > > __asan_check_store_add_4_R13
> > > > __asan_check_store_add_4_R14
> > > > __asan_check_store_add_4_R15
> > > > __asan_check_store_add_4_R8
> > > > __asan_check_store_add_4_R9
> > > > __asan_check_store_add_4_RAX
> > > > __asan_check_store_add_4_RBP
> > > > __asan_check_store_add_4_RBX
> > > > __asan_check_store_add_4_RCX
> > > > __asan_check_store_add_4_RDI
> > > > __asan_check_store_add_4_RDX
> > > > __asan_check_store_add_4_RSI
> > > > __asan_check_store_add_8_R12
> > > > __asan_check_store_add_8_R13
> > > > __asan_check_store_add_8_R14
> > > > __asan_check_store_add_8_R15
> > > > __asan_check_store_add_8_R8
> > > > __asan_check_store_add_8_R9
> > > > __asan_check_store_add_8_RAX
> > > > __asan_check_store_add_8_RBP
> > > > __asan_check_store_add_8_RBX
> > > > __asan_check_store_add_8_RCX
> > > > __asan_check_store_add_8_RDI
> > > > __asan_check_store_add_8_RDX
> > > > __asan_check_store_add_8_RSI
> > > 
> > > --
> > > 
> > > ********************
> > > ********************
> > > Failed Tests (1):
> > >   AddressSanitizer-x86_64-linux :: TestCases/Linux/interface_symbols_linux.cpp
> > > ```
> > Removed. 
> I fixed that. The issue is that macro expansion doesn't produce new lines so I had to use sed to force that. 
please clang-format this now
I expect "#  defines"

================
Comment at: compiler-rt/lib/asan/asan_shadow_defines.h:17
+#if !defined(SANITIZER_WORDSIZE)
+#  error "SANITIZER_WORDSIZE should be defined."
+#endif
----------------
vitalybuka wrote:
> kstoimenov wrote:
> > vitalybuka wrote:
> > > why can't you move this into asan_mapping.h?
> > I can't include asan_mapping.h from .S files.
> Can you do an opposite? include this one into asan_mapping and use SHADOW_OFFSET_CONST there.
> 
this comment is unreplied

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114558