[llvm] 95db373 - [OpenMP] Silence unused variable warning in release builds
Benjamin Kramer via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 4 06:21:06 PDT 2022
Author: Benjamin Kramer
Date: 2022-09-04T15:19:48+02:00
New Revision: 95db37320d3c776e98e653c0761930b6636cd268
URL: https://github.com/llvm/llvm-project/commit/95db37320d3c776e98e653c0761930b6636cd268
DIFF: https://github.com/llvm/llvm-project/commit/95db37320d3c776e98e653c0761930b6636cd268.diff
LOG: [OpenMP] Silence unused variable warning in release builds
llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp:3397:27: warning: unused variable 'SV' [-Werror]
for (const llvm::Value *SV : StoreValues)
^
Added:
Modified:
llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
Removed:
################################################################################
diff --git a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
index f776cdeecac2..92f6289087e1 100644
--- a/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
+++ b/llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp
@@ -3394,9 +3394,10 @@ OpenMPIRBuilder::createOrderedDepend(const LocationDescription &Loc,
InsertPointTy AllocaIP, unsigned NumLoops,
ArrayRef<llvm::Value *> StoreValues,
const Twine &Name, bool IsDependSource) {
- for (const llvm::Value *SV : StoreValues)
- assert(SV->getType()->isIntegerTy(64) &&
- "OpenMP runtime requires depend vec with i64 type");
+ assert(
+ llvm::all_of(StoreValues,
+ [](Value *SV) { return SV->getType()->isIntegerTy(64); }) &&
+ "OpenMP runtime requires depend vec with i64 type");
if (!updateToLocation(Loc))
return Loc.IP;
More information about the llvm-commits
mailing list