[CMake][libcxx] Support HWASan
Needs ReviewPublic
Actions

Authored by phosek on Jun 12 2022, 5:32 PM.

Details

Reviewers

ldionne

Group Reviewers

Restricted Project

Summary

This is analogous to ASan.

Diff Detail

Repository: rG LLVM Github Monorepo

Unit TestsFailed
View All

	Time	Test
	4,110 ms	libcxx CI No locale > llvm-libc++-shared-cfg-in.libcxx::modules_include.sh.cpp Script: -- : 'RUN: at line 46'; /usr/bin/c++ /home/libcxx-builder/.buildkite-agent/builds/3281ffea39a5-1/llvm-project/libcxx-ci/libcxx/test/libcxx/modules_include.sh.cpp --target=x86_64-unknown-linux-gnu -nostdinc++ -I /home/libcxx-builder/.buildkite-agent/builds/3281ffea39a5-1/llvm-project/libcxx-ci/build/generic-no-localization/include/c++/v1 -I /home/libcxx-builder/.buildkite-agent/builds/3281ffea39a5-1/llvm-project/libcxx-ci/build/generic-no-localization/include/c++/v1 -I /home/libcxx-builder/.buildkite-agent/builds/3281ffea39a5-1/llvm-project/libcxx-ci/libcxx/test/support -std=c++2b -Werror -Wall -Wextra -Wshadow -Wundef -Wno-unused-command-line-argument -Wno-attributes -Wno-pessimizing-move -Wno-c++11-extensions -Wno-noexcept-type -Wno-atomic-alignment -Wno-user-defined-literals -Wsign-compare -Wunused-variable -Wunused-parameter -Wunreachable-code -Wno-unused-local-typedef -D_LIBCPP_HAS_NO_PRAGMA_SYSTEM_HEADER -D_LIBCPP_DISABLE_AVAILABILITY -fcoroutines-ts -Werror=thread-safety -Wuser-defined-warnings -fmodules -fcxx-modules -fsyntax-only -DTEST_0

Event Timeline

phosek created this revision.Jun 12 2022, 5:32 PM

Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2022, 5:32 PM

Herald added a subscriber: mgorny. · View Herald Transcript

phosek requested review of this revision.Jun 12 2022, 5:32 PM

Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2022, 5:32 PM

Herald added a reviewer: Restricted Project. · View Herald Transcript

Herald added a subscriber: libcxx-commits. · View Herald Transcript

Harbormaster completed remote builds in B169345: Diff 436263.Jun 12 2022, 6:54 PM

I assume we should also add the ability to test libc++ that way? If so, let's add it (libcxx/utils/libcxx/test/params.py:129). How is HWAddress different from Address? Is it worth adding a CI job?

This revision now requires changes to proceed.Jun 13 2022, 8:16 AM

phosek updated this revision to Diff 436499.Jun 13 2022, 11:52 AM

Herald added a subscriber: arichardson. · View Herald TranscriptJun 13 2022, 11:52 AM

In D127608#3578073, @ldionne wrote:

I assume we should also add the ability to test libc++ that way? If so, let's add it (libcxx/utils/libcxx/test/params.py:129).

Done.