[PATCH] D69814: [Scheduling] Enable AA in PostRA Machine scheduler

[Dave Green via Phabricator via llvm-commits]

dmgreen created this revision.
dmgreen added reviewers: hfinkel, atrick, chandlerc, MatzeB, samparker.
Herald added subscribers: steven.zhang, jsji, javed.absar, hiraditya, kristof.beyls, nemanjai.
Herald added a project: LLVM.
dmgreen added a reviewer: jsji.
Herald added a subscriber: wuzish.

This adds AA to Post-RA Machine Scheduling, allowing the pass more freedom when handling memory operations.

My understanding is that this was just never done, not that it is inherently incorrect to do so. The older PostRA List scheduler already makes use of AA, it's just that the MI PostRA Scheduler was never taught to use it.

Some PowerPC tests have changed, which I don't know a lot about but look like the same number of instructions! An AArch64 test is changed which looks OK.


https://reviews.llvm.org/D69814

Files:
  llvm/lib/CodeGen/MachineScheduler.cpp
  llvm/test/CodeGen/AArch64/merge-store-dependency.ll
  llvm/test/CodeGen/PowerPC/extract-and-store.ll
  llvm/test/CodeGen/PowerPC/f128-aggregates.ll
  llvm/test/CodeGen/PowerPC/vec_conv_fp32_to_i64_elts.ll
  llvm/test/CodeGen/PowerPC/vec_conv_i16_to_fp64_elts.ll
  llvm/test/CodeGen/PowerPC/vec_conv_i64_to_fp32_elts.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69814.227723.patch
Type: text/x-patch
Size: 13616 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191104/9374388c/attachment-0001.bin>