Skip to content

Conversation

@MrSidims
Copy link
Contributor

@MrSidims MrSidims commented Feb 20, 2025

Fix warning after #127737

@llvmbot
Copy link
Member

llvmbot commented Feb 20, 2025

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

Author: Dmitry Sidorov (MrSidims)

Changes

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

1 Files Affected:

  • (modified) llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp (+4-4)
diff --git a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
index 473fcec8c4dea..3e9ce1c06850b 100644
--- a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
@@ -1053,10 +1053,10 @@ static bool buildBindlessImageINTELInst(const SPIRV::IncomingCall *Call,
     return buildOpFromWrapper(MIRBuilder, Opcode, Call,
                               GR->getSPIRVTypeID(Call->ReturnType));
 
-  auto MIB = MIRBuilder.buildInstr(Opcode)
-                 .addDef(Call->ReturnRegister)
-                 .addUse(GR->getSPIRVTypeID(Call->ReturnType))
-                 .addUse(Call->Arguments[0]);
+  MIRBuilder.buildInstr(Opcode)
+      .addDef(Call->ReturnRegister)
+      .addUse(GR->getSPIRVTypeID(Call->ReturnType))
+      .addUse(Call->Arguments[0]);
 
   return true;
 }

@MrSidims MrSidims changed the title [NFC][SPIRV] Removed unused variable [NFC][SPIR-V] Remove unused variable Feb 20, 2025
@MrSidims
Copy link
Contributor Author

MrSidims commented Feb 20, 2025

build and tests (Test SPIR-V / Lit Tests (ubuntu-22.04)) have passed, merging to fix buildbots

@MrSidims MrSidims merged commit 544a161 into llvm:main Feb 20, 2025
8 of 10 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