Skip to content

Conversation

bogner
Copy link
Contributor

@bogner bogner commented Oct 8, 2025

Fix the build with the DirectX backend enabled by using getUnversionedName() to match the updates of other similar uses of getSourceLanguage().

Fix the build with the DirectX backend enabled by using `getUnversionedName()`
to match the updates of other similar uses of `getSourceLanguage()`.
@llvmbot
Copy link
Member

llvmbot commented Oct 8, 2025

@llvm/pr-subscribers-backend-directx

Author: Justin Bogner (bogner)

Changes

Fix the build with the DirectX backend enabled by using getUnversionedName() to match the updates of other similar uses of getSourceLanguage().


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

1 Files Affected:

  • (modified) llvm/lib/Target/DirectX/DXILWriter/DXILBitcodeWriter.cpp (+1-1)
diff --git a/llvm/lib/Target/DirectX/DXILWriter/DXILBitcodeWriter.cpp b/llvm/lib/Target/DirectX/DXILWriter/DXILBitcodeWriter.cpp
index bc1a3a7995bda..82c43ff8dc352 100644
--- a/llvm/lib/Target/DirectX/DXILWriter/DXILBitcodeWriter.cpp
+++ b/llvm/lib/Target/DirectX/DXILWriter/DXILBitcodeWriter.cpp
@@ -1507,7 +1507,7 @@ void DXILBitcodeWriter::writeDICompileUnit(const DICompileUnit *N,
                                            SmallVectorImpl<uint64_t> &Record,
                                            unsigned Abbrev) {
   Record.push_back(N->isDistinct());
-  Record.push_back(N->getSourceLanguage());
+  Record.push_back(N->getSourceLanguage().getUnversionedName());
   Record.push_back(VE.getMetadataOrNullID(N->getFile()));
   Record.push_back(VE.getMetadataOrNullID(N->getRawProducer()));
   Record.push_back(N->isOptimized());

@bogner bogner merged commit 40fbe32 into llvm:main Oct 8, 2025
8 of 11 checks passed
svkeerthy pushed a commit that referenced this pull request Oct 9, 2025
Fix the build with the DirectX backend enabled by using `getUnversionedName()`
to match the updates of other similar uses of `getSourceLanguage()`.
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.

5 participants