From cc0567c0524e7f2f870d58ba85b29c7778786184 Mon Sep 17 00:00:00 2001 From: Robert Konicar Date: Thu, 9 Nov 2023 13:02:54 +0100 Subject: [PATCH] hl:lowertypedefs: Replace recursive replacer with the simple one. The recursive replacer walks only on the ops and ignores other stuff e.g. block arguments breaking the rest of the pass. --- lib/vast/Dialect/HighLevel/Transforms/LowerTypeDefs.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/vast/Dialect/HighLevel/Transforms/LowerTypeDefs.cpp b/lib/vast/Dialect/HighLevel/Transforms/LowerTypeDefs.cpp index 086b0746d8..ccd820c732 100644 --- a/lib/vast/Dialect/HighLevel/Transforms/LowerTypeDefs.cpp +++ b/lib/vast/Dialect/HighLevel/Transforms/LowerTypeDefs.cpp @@ -127,7 +127,8 @@ namespace vast::hl { mlir::AttrTypeReplacer replacer; replacer.addReplacement(conv::tc::convert_type_attr(tc)); replacer.addReplacement(conv::tc::convert_data_layout_attrs(tc)); - replacer.recursivelyReplaceElementsIn( + + replacer.replaceElementsIn( op, true /* replace attrs */, false /* replace locs */, true /* replace types */ ); };