[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