[PATCH] D87737: Add -fprofile-update={atomic,prefer-atomic,single}
David Li via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 24 23:28:15 PDT 2020
davidxl accepted this revision.
davidxl added a comment.
This revision is now accepted and ready to land.
Looks good. Makes the tsan and instrumentation interaction also cleaner.
================
Comment at: clang/test/CodeGen/code-coverage-tsan.c:1
-/// -fsanitize=thread requires the (potentially concurrent) counter updates to be atomic.
-// RUN: %clang_cc1 %s -triple x86_64 -emit-llvm -fsanitize=thread -femit-coverage-notes -femit-coverage-data \
+/// -fprofile-update=atmomic (implied by -fsanitize=thread) requires the
+/// (potentially concurrent) counter updates to be atomic.
----------------
Fix typo: atmomic
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87737/new/
https://reviews.llvm.org/D87737
More information about the cfe-commits
mailing list