Skip to content

Commit dae31e5

Browse files
Stylie777llvmbot
authored andcommitted
Revert "[AArch64] Enable Spillage Copy Elimination by default" (llvm#211572)
Reverts llvm#186093 Issue llvm#206839 shows that EliminateSpillageCopies is not currently ready to be enabled by default on AArch64. llvm#207169 was raised to fix the issue, but concerns were raised around complexity of the function and change. Until a suitable solution can be found, EliminateSpillageCopy will be disabled on AArch64 by default. For those who wish to still use the pass, `-enable-spill-copy-elim=true` will ensure the pass is run. (cherry picked from commit debd8a7)
1 parent 2a0942d commit dae31e5

3 files changed

Lines changed: 30 additions & 424 deletions

File tree

llvm/lib/Target/AArch64/AArch64Subtarget.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@ class AArch64Subtarget final : public AArch64GenSubtargetInfo {
160160
bool enableMachineScheduler() const override { return true; }
161161
bool enablePostRAScheduler() const override { return usePostRAScheduler(); }
162162
bool enableSubRegLiveness() const override { return EnableSubregLiveness; }
163-
bool enableSpillageCopyElimination() const override { return true; }
164163

165164
bool enableMachinePipeliner() const override;
166165
bool useDFAforSMS() const override { return false; }

0 commit comments

Comments
 (0)