Skip to content

Conversation

@sivadeilra
Copy link
Contributor

#146121 caused a build warning in a post-CI job. The fix should be trivial; provide an extra initializer element when initializing a class, for the additional field.

@llvmbot
Copy link
Member

llvmbot commented Jul 7, 2025

@llvm/pr-subscribers-debuginfo

Author: None (sivadeilra)

Changes

#146121 caused a build warning in a post-CI job. The fix should be trivial; provide an extra initializer element when initializing a class, for the additional field.


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

1 Files Affected:

  • (modified) llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp (+2-1)
diff --git a/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp b/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
index 65a873553b9ac..bc74daf983e40 100644
--- a/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
+++ b/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
@@ -3572,7 +3572,8 @@ void CodeViewDebug::collectDebugInfoForJumpTables(const MachineFunction *MF,
                             BaseOffset,
                             Branch,
                             MF->getJTISymbol(JumpTableIndex, MMI->getContext()),
-                            JTE.MBBs.size()};
+                            JTE.MBBs.size(),
+                            {}};
         for (const auto &MBB : JTE.MBBs)
           CVJTI.Cases.push_back(MBB->getSymbol());
         CurFn->JumpTables.push_back(std::move(CVJTI));

@dpaoliello dpaoliello merged commit 65de879 into llvm:main Jul 7, 2025
11 of 12 checks passed
bonsthie pushed a commit to bonsthie/llvm-project that referenced this pull request Jul 22, 2025
llvm#146121 caused a build warning in a post-CI job. The fix should be
trivial; provide an extra initializer element when initializing a class,
for the additional field.
@sivadeilra sivadeilra deleted the user/ardavis/fix-codeview-warning-jumps branch August 7, 2025 16:54
steven-studio pushed a commit to steven-studio/llvm-project that referenced this pull request Sep 11, 2025
llvm#146121 caused a build warning in a post-CI job. The fix should be
trivial; provide an extra initializer element when initializing a class,
for the additional field.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants