[PATCH] D142699: [AsmParser] Avoid instantiating LLVMContext if not needed. NFC

[Jannik Silvanus via Phabricator via llvm-commits]

jsilvanus added inline comments.


================
Comment at: llvm/lib/AsmParser/Parser.cpp:36
+  return LLParser(F.getBuffer(), SM, Err, M, Index, Context, Slots)
       .Run(UpgradeDebugInfo, DataLayoutCallback);
 }
----------------
Instead of duplicating the call, maybe you can do something like

```
std::optional<LLVMContext> OptContext;
return LLParser(F.getBuffer(), SM, Err, M, Index,
                M ? M->getContext() : OptContext.emplace(), Slots)
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142699/new/

https://reviews.llvm.org/D142699