[llvm] [SPIRV] Silence unused variable warnings (PR #95492)
Chris B via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 13 18:09:17 PDT 2024
https://github.com/llvm-beanz created https://github.com/llvm/llvm-project/pull/95492
This change marks a few variable declarations as [[maybe_unused]] to silence unused variable warnings.
../llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp
>From cffee9c0bee88011e719ef175a14fcc09ffe5d83 Mon Sep 17 00:00:00 2001
From: Chris Bieneman <chris.bieneman at me.com>
Date: Thu, 13 Jun 2024 19:55:33 -0500
Subject: [PATCH] [SPIRV] Silence unused variable warnings
This change marks a few variable declarations as [[maybe_unused]] to
silence unused variable warnings.
../llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp
---
.../Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp | 2 +-
llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp b/llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp
index 7f5f7d0b1e4dc..25e285e35f933 100644
--- a/llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp
+++ b/llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp
@@ -138,7 +138,7 @@ ConvergenceRegion::ConvergenceRegion(
SmallPtrSet<BasicBlock *, 8> &&Blocks, SmallPtrSet<BasicBlock *, 2> &&Exits)
: DT(DT), LI(LI), ConvergenceToken(ConvergenceToken), Entry(Entry),
Exits(std::move(Exits)), Blocks(std::move(Blocks)) {
- for (auto *BB : this->Exits)
+ for ([[maybe_unused]] auto *BB : this->Exits)
assert(this->Blocks.count(BB) != 0);
assert(this->Blocks.count(this->Entry) != 0);
}
diff --git a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
index 2a515359bdd4f..c14e5098be711 100644
--- a/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
@@ -2418,7 +2418,7 @@ Type *parseBuiltinCallArgumentBaseType(const StringRef DemangledCall,
if (hasBuiltinTypePrefix(TypeStr)) {
// OpenCL builtin types in demangled call strings have the following format:
// e.g. ocl_image2d_ro
- bool IsOCLBuiltinType = TypeStr.consume_front("ocl_");
+ [[maybe_unused]] bool IsOCLBuiltinType = TypeStr.consume_front("ocl_");
assert(IsOCLBuiltinType && "Invalid OpenCL builtin prefix");
// Check if this is pointer to a builtin type and not just pointer
More information about the llvm-commits
mailing list