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

[Kirill Stoimenov via Phabricator via llvm-commits]

kstoimenov added inline comments.


================
Comment at: compiler-rt/lib/asan/asan_interface.inc:184
+
+// clang-format off
+#if defined(__x86_64__)
----------------
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. 


================
Comment at: compiler-rt/lib/asan/asan_interface.inc:184
+
+// clang-format off
+#if defined(__x86_64__)
----------------
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. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114558