[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; }
+
+  class TopLevelInstParserRegistry {
+  public:
+    template <std::byte opCode>
+    static parsed_inst_t parseInstrWithOpCode(OpBuilder &,
+                                              ExpressionParser &) = delete;
----------------
lforg37 wrote:

I didn't know that issue, thanks for bringing it up. IIUC this will not pose problem as we're going to define the specialization in th esame enclosing namespace. 
Subparser will be able to be defined in their own namespaces.

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