Merge pull request #15047 from ethereum/remove_wordSizeTransform

Remove wordSizeTransform left-over.
ethereum · Apr 22, 2024 · 272892e · 272892e
2 parents 2e8cd3f + cc27468
commit 272892e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/test/libyul/YulOptimizerTest.cpp b/test/libyul/YulOptimizerTest.cpp
@@ -81,8 +81,7 @@ TestCase::TestResult YulOptimizerTest::run(std::ostream& _stream, std::string co
 	auto const printed = (m_object->subObjects.empty() ? AsmPrinter{ *m_dialect }(*m_object->code) : m_object->toString(m_dialect));
 
 	// Re-parse new code for compilability
-	// TODO: support for wordSizeTransform which needs different input and output dialects
-	if (m_optimizerStep != "wordSizeTransform" && !std::get<0>(parse(_stream, _linePrefix, _formatted, printed)))
+	if (!std::get<0>(parse(_stream, _linePrefix, _formatted, printed)))
 	{
 		util::AnsiColorized(_stream, _formatted, {util::formatting::BOLD, util::formatting::CYAN})
 			<< _linePrefix << "Result after the optimiser:" << std::endl;

diff --git a/test/libyul/YulOptimizerTestCommon.cpp b/test/libyul/YulOptimizerTestCommon.cpp
@@ -417,13 +417,12 @@ std::string YulOptimizerTestCommon::randomOptimiserStep(unsigned _seed)
 		if (count == idx)
 		{
 			std::string optimiserStep = step.first;
-			// Do not fuzz mainFunction and wordSizeTransform
-			// because they do not preserve yul code semantics.
+			// Do not fuzz mainFunction
+			// because it does not preserve yul code semantics.
 			// Do not fuzz reasoning based simplifier because
 			// it can sometimes drain memory.
 			if (
-				optimiserStep == "mainFunction" ||
-				optimiserStep == "wordSizeTransform"
+				optimiserStep == "mainFunction"
 			)
 				// "Fullsuite" is fuzzed roughly four times more frequently than
 				// other steps because of the filtering in place above.