Skip to content

Conversation

@ellishg
Copy link
Contributor

@ellishg ellishg commented Oct 30, 2025

The --debug-info-correlate flag was removed in #165289, but I must have forgotten this test. Replace with --profile-correlate=debug-info to fix.

@ellishg ellishg requested a review from ZequanWu October 30, 2025 16:40
@ellishg ellishg enabled auto-merge (squash) October 30, 2025 16:40
@llvmbot llvmbot added compiler-rt PGO Profile Guided Optimizations labels Oct 30, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 30, 2025

@llvm/pr-subscribers-pgo

Author: Ellis Hoag (ellishg)

Changes

The --debug-info-correlate flag was removed in #165289, but I must have forgotten this test. Replace with --profile-correlate=debug-info to fix.


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

1 Files Affected:

  • (modified) compiler-rt/test/profile/Linux/instrprof-debug-info-correlate-warnings.c (+1-1)
diff --git a/compiler-rt/test/profile/Linux/instrprof-debug-info-correlate-warnings.c b/compiler-rt/test/profile/Linux/instrprof-debug-info-correlate-warnings.c
index 5069c6340b64f..25022f241a6d2 100644
--- a/compiler-rt/test/profile/Linux/instrprof-debug-info-correlate-warnings.c
+++ b/compiler-rt/test/profile/Linux/instrprof-debug-info-correlate-warnings.c
@@ -1,6 +1,6 @@
 // Disable full debug info and verify that we get warnings during merging
 
-// RUN: %clang_pgogen -o %t -gline-tables-only -mllvm --debug-info-correlate -mllvm --disable-vp=true %S/../Inputs/instrprof-debug-info-correlate-main.cpp %S/../Inputs/instrprof-debug-info-correlate-foo.cpp
+// RUN: %clang_pgogen -o %t -gline-tables-only -mllvm --profile-correlate=debug-info -mllvm --disable-vp=true %S/../Inputs/instrprof-debug-info-correlate-main.cpp %S/../Inputs/instrprof-debug-info-correlate-foo.cpp
 // RUN: env LLVM_PROFILE_FILE=%t.proflite %run %t
 // RUN: llvm-profdata merge -o %t.profdata --debug-info=%t %t.proflite --max-debug-info-correlation-warnings=2 2>&1 >/dev/null | FileCheck %s --check-prefixes=CHECK,LIMIT --implicit-check-not=warning
 // RUN: llvm-profdata merge -o %t.profdata --debug-info=%t %t.proflite --max-debug-info-correlation-warnings=0 2>&1 >/dev/null | FileCheck %s --check-prefixes=CHECK,NOLIMIT --implicit-check-not=warning

@ellishg ellishg disabled auto-merge October 30, 2025 16:41
@ellishg ellishg merged commit 88cee4c into llvm:main Oct 30, 2025
8 of 12 checks passed
luciechoi pushed a commit to luciechoi/llvm-project that referenced this pull request Nov 1, 2025
The `--debug-info-correlate` flag was removed in
llvm#165289, but I must have
forgotten this test. Replace with `--profile-correlate=debug-info` to
fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compiler-rt PGO Profile Guided Optimizations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants