[all-commits] [llvm/llvm-project] 151066: [asan] Add missing #include of sanitizer_platform.h
Roland McGrath via All-commits
all-commits at lists.llvm.org
Wed Jan 26 18:19:25 PST 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 1510668fb007c448cefb2190bf0558c63cdd5382
https://github.com/llvm/llvm-project/commit/1510668fb007c448cefb2190bf0558c63cdd5382
Author: Roland McGrath <mcgrathr at google.com>
Date: 2022-01-26 (Wed, 26 Jan 2022)
Changed paths:
M compiler-rt/lib/asan/asan_mapping.h
Log Message:
-----------
[asan] Add missing #include of sanitizer_platform.h
The "asan/asan_mapping.h" header relies on sanitizer_platform.h
macros, but doesn't directly include the header. All the existing
uses until recently happened to be in places where some other header
had indirectly included sanitizer_platform.h first. The addition of
asan_rtl_x86_64.S was the first place to use "asan/asan_mapping.h"
alone. It so happens that its uses of the macros make having no
macros defined equivalent to SANITIZER_LINUX, so this did not affect
Linux builds. But the assembly constants in asan_rtl_x86_64.S were
wrong for Fuchsia when SANITIZER_FUCHSIA was not properly defined.
Reviewed By: vitalybuka
Differential Revision: https://reviews.llvm.org/D118296
More information about the All-commits
mailing list