[flang-commits] [flang] [mlir] [flang][mlir] Add llvm.ident metadata when compiling with flang (PR #102506)

Christian Ulmann via flang-commits flang-commits at lists.llvm.org
Mon Aug 12 07:35:30 PDT 2024


================
@@ -114,6 +114,22 @@ mlir::LLVM::TargetFeaturesAttr fir::getTargetFeatures(mlir::ModuleOp mod) {
   return {};
 }
 
+void fir::setIdent(mlir::ModuleOp mod, llvm::StringRef ident) {
+  if (ident.empty())
+    return;
+
+  auto *ctx = mod.getContext();
----------------
Dinistro wrote:

```suggestion
  mlir::MLIRContext *ctx = mod.getContext();
```
Nit: Only use auto when the type is given on the RHS or when it is too complex to write out.

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


More information about the flang-commits mailing list