[Mlir-commits] [mlir] [MLIR][WasmSSA] Instruction parser refactoring of WasmSSA importer (PR #195500)

[Luc Forget]

================
@@ -444,6 +420,15 @@ class ExpressionParser {
   template <typename OpToCreate>
   parsed_inst_t parseBlockLikeOp(OpBuilder &);
 
+  std::optional<Location> getCurrentOpLoc() { return currentOpLoc; }
----------------
lforg37 wrote:

I'm not even sure we need the assert as the optional dereferencement operator will error out anyway.

But given how it is used it makes sense indeed, will do. Thanks. 


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