Skip to content

Commit e178496

Browse files
author
Justin Michaud
committed
Remove loop strength reduction because WasmAddress broke it.
https://bugs.webkit.org/show_bug.cgi?id=243178 Reviewed by Yusuke Suzuki. When WasmAddress was added, it broke this pass. We should eventually fix it, but for now we should remove it since it never actually gets applied. * Source/JavaScriptCore/Sources.txt: * Source/JavaScriptCore/b3/B3Generate.cpp: (JSC::B3::generateToAir): * Source/JavaScriptCore/b3/B3ReduceLoopStrength.cpp: Removed. * Source/JavaScriptCore/b3/B3ReduceLoopStrength.h: Removed. * Source/JavaScriptCore/b3/testb3.h: * Source/JavaScriptCore/b3/testb3_1.cpp: (run): * Source/JavaScriptCore/b3/testb3_8.cpp: (testFastForwardCopy32): Deleted. (testByteCopyLoop): Deleted. (testByteCopyLoopStartIsLoopDependent): Deleted. (testByteCopyLoopBoundIsLoopDependent): Deleted. (addCopyTests): Deleted. Canonical link: https://commits.webkit.org/256507@main
1 parent abb1790 commit e178496

7 files changed

Lines changed: 0 additions & 846 deletions

File tree

‎Source/JavaScriptCore/Sources.txt‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,6 @@ b3/B3PhiChildren.cpp
165165
b3/B3Procedure.cpp
166166
b3/B3PureCSE.cpp
167167
b3/B3ReduceDoubleToFloat.cpp
168-
b3/B3ReduceLoopStrength.cpp
169168
b3/B3ReduceStrength.cpp
170169
b3/B3SSACalculator.cpp
171170
b3/B3SlotBaseValue.cpp

‎Source/JavaScriptCore/b3/B3Generate.cpp‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@
4646
#include "B3OptimizeAssociativeExpressionTrees.h"
4747
#include "B3Procedure.h"
4848
#include "B3ReduceDoubleToFloat.h"
49-
#include "B3ReduceLoopStrength.h"
5049
#include "B3ReduceStrength.h"
5150
#include "B3Validate.h"
5251
#include "CompilerTimingScope.h"
@@ -92,7 +91,6 @@ void generateToAir(Procedure& procedure)
9291
eliminateCommonSubexpressions(procedure);
9392
eliminateDeadCode(procedure);
9493
inferSwitches(procedure);
95-
reduceLoopStrength(procedure);
9694
if (Options::useB3TailDup())
9795
duplicateTails(procedure);
9896
fixSSA(procedure);

0 commit comments

Comments
 (0)