[PATCH] D125377: [AArch64] Order STP Q's by ascending address

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 12 08:26:36 PDT 2022


fhahn added a comment.

In D125377#3508288 <https://reviews.llvm.org/D125377#3508288>, @dmgreen wrote:

> In order sounds sensible to me. We may need a subtarget feature for this, it depends on what the Apple folks think, but we can always add one later if needed.

I *think* it should be fine, but I can double-check.



================
Comment at: llvm/lib/Target/AArch64/AArch64TargetMachine.cpp:484
       // pseudos then (v. addPreSched2()).
-      ScheduleDAGMI *DAG = createGenericSchedPostRA(C);
       DAG->addMutation(createAArch64MacroFusionDAGMutation());
       return DAG;
----------------
Might make sense to move the code from `AArch64MacroFusion.cpp` to the new `"AArch64MachineScheduler.h"`


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125377/new/

https://reviews.llvm.org/D125377



More information about the llvm-commits mailing list