Skip to content

Conversation

maarquitos14
Copy link
Contributor

@maarquitos14 maarquitos14 commented Aug 5, 2025

Fix the buildbot failure caused by merging #149522.

@llvmbot
Copy link
Member

llvmbot commented Aug 5, 2025

@llvm/pr-subscribers-backend-spir-v

Author: Marcos Maronas (maarquitos14)

Changes

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

1 Files Affected:

  • (modified) llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp (+1)
diff --git a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp
index 000e2d7105c8d..ab06fc0b5ff31 100644
--- a/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVModuleAnalysis.cpp
@@ -1874,6 +1874,7 @@ void addInstrRequirements(const MachineInstr &MI,
   case SPIRV::OpCopyMemorySized: {
     Reqs.addCapability(SPIRV::Capability::Addresses);
     // TODO: Add UntypedPointersKHR when implemented.
+    break;
   }
 
   default:

@MrSidims
Copy link
Contributor

MrSidims commented Aug 5, 2025

Merging ahead of CI to fix buildbots.

@MrSidims MrSidims merged commit fd07d90 into llvm:main Aug 5, 2025
10 of 12 checks passed
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