Skip to content

Commit 708cda8

Browse files
committed
Procude as much warnings as possible
1 parent 3f347a7 commit 708cda8

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

mlir/lib/Target/LLVMIR/ModuleImport.cpp

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -530,11 +530,12 @@ LogicalResult ModuleImport::convertModuleFlagsMetadata() {
530530
for (const auto [behavior, key, val] : llvmModuleFlags) {
531531
// Currently only supports most common: int constant values.
532532
auto *constInt = llvm::mdconst::dyn_extract<llvm::ConstantInt>(val);
533-
if (!constInt)
534-
return emitWarning(mlirModule.getLoc())
535-
<< "unsupported module flag value: "
536-
<< diagMD(val, llvmModule.get())
537-
<< ", only constant integer currently supported";
533+
if (!constInt) {
534+
emitWarning(mlirModule.getLoc())
535+
<< "unsupported module flag value: " << diagMD(val, llvmModule.get())
536+
<< ", only constant integer currently supported";
537+
continue;
538+
}
538539

539540
moduleFlags.push_back(builder.getAttr<ModuleFlagAttr>(
540541
convertModFlagBehaviorFromLLVM(behavior),

0 commit comments

Comments
 (0)