[clang] [profile] runtime counter relocation is needed on windows-msvc targets (PR #127858)

via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 19 11:34:54 PST 2025


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 1841bcd5d16310c052c424dec3bcf2b703badd40 843f78e9826b037a7bfb0913728a11eb249f4181 --extensions c,cpp -- clang/lib/Driver/ToolChains/Clang.cpp clang/test/Driver/fprofile-continuous.c
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/clang/lib/Driver/ToolChains/Clang.cpp b/clang/lib/Driver/ToolChains/Clang.cpp
index 3f76d034ee..e7acc3f552 100644
--- a/clang/lib/Driver/ToolChains/Clang.cpp
+++ b/clang/lib/Driver/ToolChains/Clang.cpp
@@ -795,7 +795,8 @@ static void addPGOAndCoverageFlags(const ToolChain &TC, Compilation &C,
     else {
       CmdArgs.push_back("-fprofile-continuous");
       // Platforms that require a bias variable:
-      if (T.isOSBinFormatELF() || T.isOSAIX() || T.isKnownWindowsMSVCEnvironment()) {
+      if (T.isOSBinFormatELF() || T.isOSAIX() ||
+          T.isKnownWindowsMSVCEnvironment()) {
         CmdArgs.push_back("-mllvm");
         CmdArgs.push_back("-runtime-counter-relocation");
       }

``````````

</details>


https://github.com/llvm/llvm-project/pull/127858


More information about the cfe-commits mailing list