Skip to content

Conversation

@ellishg
Copy link
Contributor

@ellishg ellishg commented Jan 23, 2025

XFAIL llvm/test/tools/llvm-profdata/general.proftext after it was accidentally broken by #105915. I will follow up to get this fixed.

@llvmbot llvmbot added the PGO Profile Guided Optimizations label Jan 23, 2025
@llvmbot
Copy link
Member

llvmbot commented Jan 23, 2025

@llvm/pr-subscribers-pgo

Author: Ellis Hoag (ellishg)

Changes

XFAIL llvm/test/tools/llvm-profdata/general.proftext after it was accidentally broken by #105915. I will follow up to get this fixed.


Full diff: https://github.com/llvm/llvm-project/pull/124165.diff

1 Files Affected:

  • (modified) llvm/test/tools/llvm-profdata/general.proftext (+2)
diff --git a/llvm/test/tools/llvm-profdata/general.proftext b/llvm/test/tools/llvm-profdata/general.proftext
index 89762f2540f6a6..ca532f9a37116d 100644
--- a/llvm/test/tools/llvm-profdata/general.proftext
+++ b/llvm/test/tools/llvm-profdata/general.proftext
@@ -1,3 +1,5 @@
+# FIXME: Somehow this is failing on windows after https://github.com/llvm/llvm-project/pull/105915
+# XFAIL: system-windows
 # RUN: llvm-profdata merge -sparse=true %s -o %t.profdata
 
 # RUN: llvm-profdata merge -sparse=false %s -o %t.profdata.dense

@ellishg ellishg merged commit 0290693 into llvm:main Jan 23, 2025
7 of 8 checks passed
@ellishg ellishg deleted the fix-profdata-summary branch January 23, 2025 18:02
ellishg added a commit that referenced this pull request Jan 23, 2025
The detailed summary format was changed in
#105915 which broke
`llvm/test/tools/llvm-profdata/general.proftext` (XFAILed in
#124165). Apparently the
behavior of `%lu` is different between Linux and Windows, so I reverted
back to using `<<` style formats.
github-actions bot pushed a commit to arm/arm-toolchain that referenced this pull request Jan 23, 2025
…24169)

The detailed summary format was changed in
llvm/llvm-project#105915 which broke
`llvm/test/tools/llvm-profdata/general.proftext` (XFAILed in
llvm/llvm-project#124165). Apparently the
behavior of `%lu` is different between Linux and Windows, so I reverted
back to using `<<` style formats.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PGO Profile Guided Optimizations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants