[Mlir-commits] [mlir] [mlir] Fix possible null dereference during error logging (PR #157455)

[Mehdi Amini]

================
@@ -405,7 +405,8 @@ std::unique_ptr<MPIImplTraits> MPIImplTraits::get(ModuleOp &moduleOp) {
     return std::make_unique<OMPIImplTraits>(moduleOp);
   if (!strAttr || strAttr.getValue() != "MPICH")
     moduleOp.emitWarning() << "Unknown \"MPI:Implementation\" value in DLTI ("
-                           << strAttr.getValue() << "), defaulting to MPICH";
+                           << (strAttr ? strAttr.getValue() : "<NULL>")
----------------
joker-eph wrote:

Can we just do `<< strAttr ` ?

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