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

[Max Graey]

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

How about to simplify and ged rid of optional?
```cpp
Location getCurrentOpLoc() const {
  assert(currentOpLoc && "expected current opcode location to be set");
  return *currentOpLoc;
}
```

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