[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