Skip to content

port 5b4819e to release#159209

Merged
dyung merged 1 commit into
llvm:release/21.xfrom
dwblaikie:port_5b4819e_to_release
Oct 6, 2025
Merged

port 5b4819e to release#159209
dyung merged 1 commit into
llvm:release/21.xfrom
dwblaikie:port_5b4819e_to_release

Conversation

@dwblaikie

@dwblaikie dwblaikie commented Sep 16, 2025

Copy link
Copy Markdown
Contributor

Applies the fix on top of the backport made in #156664

@dwblaikie dwblaikie closed this Sep 16, 2025
@dwblaikie dwblaikie reopened this Sep 16, 2025
@dwblaikie dwblaikie changed the base branch from main to release/21.x September 16, 2025 22:54
@dwblaikie dwblaikie marked this pull request as ready for review September 16, 2025 22:55
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Sep 16, 2025
@llvmbot

llvmbot commented Sep 16, 2025

Copy link
Copy Markdown
Member

@llvm/pr-subscribers-clang

Author: David Blaikie (dwblaikie)

Changes

Applies the fix on top of the backport made in #156664


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

1 Files Affected:

  • (modified) clang/test/CodeGenCXX/debug-info-structured-binding.cpp (+1-1)
diff --git a/clang/test/CodeGenCXX/debug-info-structured-binding.cpp b/clang/test/CodeGenCXX/debug-info-structured-binding.cpp
index 4a4a4d8bdfaad..8032ce85c9e25 100644
--- a/clang/test/CodeGenCXX/debug-info-structured-binding.cpp
+++ b/clang/test/CodeGenCXX/debug-info-structured-binding.cpp
@@ -10,7 +10,7 @@
 // CHECK: getelementptr inbounds nuw %struct.A, ptr {{.*}}, i32 0, i32 1, !dbg ![[Y1_DEBUG_LOC:[0-9]+]]
 // CHECK: getelementptr inbounds nuw %struct.A, ptr {{.*}}, i32 0, i32 1, !dbg ![[Y2_DEBUG_LOC:[0-9]+]]
 // CHECK: load ptr, ptr %z2, {{.*}}!dbg ![[Z2_DEBUG_LOC:[0-9]+]]
-// CHECK: getelementptr inbounds [2 x i32], ptr {{.*}}, i64 0, i64 1, !dbg ![[A2_DEBUG_LOC:[0-9]+]]
+// CHECK: getelementptr inbounds [2 x i32], ptr {{.*}}, i{{64|32}} 0, i{{64|32}} 1, !dbg ![[A2_DEBUG_LOC:[0-9]+]]
 // CHECK: getelementptr inbounds nuw { i32, i32 }, ptr {{.*}}, i32 0, i32 1, !dbg ![[C2_DEBUG_LOC:[0-9]+]]
 // CHECK: extractelement <2 x i32> {{.*}}, i32 1, !dbg ![[V2_DEBUG_LOC:[0-9]+]]
 // CHECK: ![[VAR_0]] = !DILocalVariable(name: "a"

@nikic nikic added this to the LLVM 21.x Release milestone Sep 17, 2025
@github-project-automation github-project-automation Bot moved this to Needs Triage in LLVM Release Status Sep 17, 2025
@dwblaikie dwblaikie requested a review from tru September 19, 2025 21:17
@tstellar tstellar moved this from Needs Triage to Needs Test Fix in LLVM Release Status Sep 23, 2025
@tstellar

Copy link
Copy Markdown
Contributor

Are the test failures legitimate?

@dwblaikie

dwblaikie commented Sep 23, 2025

Copy link
Copy Markdown
Contributor Author

Are the test failures legitimate?

sync and rerun seems to have cleared them up

@github-project-automation github-project-automation Bot moved this from Needs Test Fix to Needs Merge in LLVM Release Status Sep 25, 2025
@dyung dyung force-pushed the port_5b4819e_to_release branch from 31b750f to 4e9d1b6 Compare October 1, 2025 14:41
@dyung

dyung commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

Hi @dwblaikie, before I merge your change into the release branch, can you confirm that the premerge failures in libcxx were not caused by your change here?

@dwblaikie

Copy link
Copy Markdown
Contributor Author

Hi @dwblaikie, before I merge your change into the release branch, can you confirm that the premerge failures in libcxx were not caused by your change here?

So far as I can tell, yeah, they seem unrelated.

@ldionne - any idea what's going on with libcxx presubmit checks here? Failures like "Error: Failed to remove "/scratch/powerllvm/cpap8006/llvm-project/libcxx-ci" (unlinkat /scratch/powerllvm/cpap8006/llvm-project/libcxx-ci/.ci/all_requirements.txt: permission denied)" - https://buildkite.com/llvm-project/libcxx-ci/builds/83843

@dyung dyung force-pushed the port_5b4819e_to_release branch from 4e9d1b6 to 9158b87 Compare October 6, 2025 06:31
Fix for the port of 665e875 to release/21.x in PR156664
@dyung dyung force-pushed the port_5b4819e_to_release branch from 9158b87 to 05b5090 Compare October 6, 2025 14:43
@dyung dyung merged commit 05b5090 into llvm:release/21.x Oct 6, 2025
2 of 3 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Merge to Done in LLVM Release Status Oct 6, 2025
@github-actions

github-actions Bot commented Oct 6, 2025

Copy link
Copy Markdown

@dwblaikie (or anyone else). If you would like to add a note about this fix in the release notes (completely optional). Please reply to this comment with a one or two sentence description of the fix. When you are done, please add the release:note label to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang Clang issues not falling into any other category

Projects

Development

Successfully merging this pull request may close these issues.

5 participants