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

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114558