[PATCH] D159137: [AIX] Fix Link Issue when `-fprofile-update=[atomic|prefer-atomic]` is in Effect

Qiongsi Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 16:49:42 PDT 2023


qiongsiwu1 marked 2 inline comments as done.
qiongsiwu1 added inline comments.


================
Comment at: clang/test/Driver/fprofile-update.c:17
+// AIX specific tests
+// RUN: %if system-aix %{ %clang -### %s -fprofile-generate -fprofile-update=atomic 2>&1 | FileCheck %s --check-prefix=AIX %}
+// RUN: %if system-aix %{ %clang -### %s -fprofile-generate -fprofile-update=prefer-atomic 2>&1 | FileCheck %s --check-prefix=AIX %}
----------------
MaskRay wrote:
> The idiom is to use `--target=...`. This driver decision is not dependent on what OS the host runs, so `%if system-aix` should be avoided.
Ah yes thanks for the feedback. Fixed! 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D159137/new/

https://reviews.llvm.org/D159137



More information about the cfe-commits mailing list