[Mlir-commits] [mlir] [TOSA] Add StatefulOps to TOSA Dialect (PR #66843)

[Adrian Kuegel]

================
@@ -398,8 +404,12 @@ struct TosaValidation : public tosa::impl::TosaValidationBase<TosaValidation> {
     }
   }
 
+  bool CheckVariable(Operation *op);
+  bool CheckVariableReadOrWrite(Operation *op);
+
   SmallVector<std::function<LogicalResult(Operation *)>> const_checkers;
   tosa_level_t tosa_level;
+  std::unordered_map<std::string, mlir::Type> variables_map;
----------------
akuegel wrote:

I already tried that just replacing StringAttr* in the DenseHashMap with StringAttr, and accessing the key without "&" dereference, makes the test pass with --config=asan. So these small changes should be enough :)

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