This is an archive of the discontinued LLVM Phabricator instance.

Differential D119588

[sanitizers] Add missing header to fix Fuchsia builds
ClosedPublic

Authored by paulkirth on Feb 11 2022, 1:24 PM.

Details

Reviewers
phosek
mcgrathr
leonardchan
haowei
abrachet
mysterymath
Commits
rG45bddf54e240: [sanitizers] Add missing header to fix Fuchsia builds
Summary

Adds missing header to fix sanitizer builds for Fuchsia.

Diff Detail

Event Timeline

paulkirth created this revision.Feb 11 2022, 1:24 PM
Herald added a subscriber: abrachet. · View Herald TranscriptFeb 11 2022, 1:24 PM
paulkirth requested review of this revision.Feb 11 2022, 1:24 PM
paulkirth added reviewers: abrachet, mysterymath.
abrachet accepted this revision.Feb 11 2022, 1:26 PM
This revision is now accepted and ready to land.Feb 11 2022, 1:26 PM
This revision was landed with ongoing or failed builds.Feb 11 2022, 1:58 PM
Closed by commit rG45bddf54e240: [sanitizers] Add missing header to fix Fuchsia builds (authored by paulkirth). · Explain Why
This revision was automatically updated to reflect the committed changes.
paulkirth added a commit: rG45bddf54e240: [sanitizers] Add missing header to fix Fuchsia builds.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2022, 1:58 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
Harbormaster completed remote builds in B149096: Diff 408032.Feb 11 2022, 2:31 PM
paulkirth mentioned this in D119570: hwasan: fix up includes.Feb 11 2022, 3:05 PM

Revision Contents

PathSize
compiler-rt/
lib/
sanitizer_common/
1 line

Diff 408032

compiler-rt/lib/sanitizer_common/sanitizer_coverage_fuchsia.cpp

Show All 27 Lines
#include "sanitizer_platform.h" #include "sanitizer_platform.h"
#if SANITIZER_FUCHSIA #if SANITIZER_FUCHSIA
#include <zircon/process.h> #include <zircon/process.h>
#include <zircon/sanitizer.h> #include <zircon/sanitizer.h>
#include <zircon/syscalls.h> #include <zircon/syscalls.h>
#include "sanitizer_atomic.h" #include "sanitizer_atomic.h"
#include "sanitizer_common.h" #include "sanitizer_common.h"
#include "sanitizer_interface_internal.h"
Lint: Pre-merge checks
Inline Actions

clang-format: please reformat the code

-#include "sanitizer_interface_internal.h"
-#include "sanitizer_internal_defs.h"
-#include "sanitizer_symbolizer_fuchsia.h"
+#  include "sanitizer_interface_internal.h"
+#  include "sanitizer_internal_defs.h"
+#  include "sanitizer_symbolizer_fuchsia.h"
Lint: Pre-merge checks: clang-format: please reformat the code ``` -#include "sanitizer_interface_internal.h" -#include…
#include "sanitizer_internal_defs.h" #include "sanitizer_internal_defs.h"
#include "sanitizer_symbolizer_fuchsia.h" #include "sanitizer_symbolizer_fuchsia.h"
using namespace __sanitizer; using namespace __sanitizer;
namespace __sancov { namespace __sancov {
namespace { namespace {
▲ Show 20 Lines • Show All 210 Lines • Show Last 20 Lines