[Mlir-commits] [mlir] [MLIR][LLVMIR] Add module flags support (PR #130679)

[Tobias Gysi]

================
@@ -517,6 +517,38 @@ void ModuleImport::addDebugIntrinsic(llvm::CallInst *intrinsic) {
   debugIntrinsics.insert(intrinsic);
 }
 
+LogicalResult ModuleImport::convertModuleFlagsMetadata() {
+  SmallVector<llvm::Module::ModuleFlagEntry, 4> llvmModuleFlags;
----------------
gysit wrote:

```suggestion
  SmallVector<llvm::Module::ModuleFlagEntry> llvmModuleFlags;
```
nit: is there a specific reason for the 4, if not the convention is to not use an explicit size hint.

https://github.com/llvm/llvm-project/pull/130679