[PATCH] D111551: [AArch64] Split out processor/tuning features

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 14 07:18:08 PDT 2021


david-arm updated this revision to Diff 379711.
david-arm edited the summary of this revision.
david-arm added a comment.

Hi @dmgreen, it's quite difficult to write tests to prove that the tuning has any effect, because we can only test the side-effects of the tuning features. These mostly involve different scheduling choices. It took quite a while write a test, which I did by deliberately removing lots of tuning features from CPUs and looking for tests that broke. The test I have added basically ensures we schedule instructions differently. However, if you can think of a better test I'm willing to consider it!


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

https://reviews.llvm.org/D111551

Files:
  llvm/lib/Target/AArch64/AArch64.td
  llvm/lib/Target/AArch64/AArch64Subtarget.h
  llvm/lib/Target/AArch64/AArch64SystemOperands.td
  llvm/test/CodeGen/AArch64/misched-fusion-addr-tune.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D111551.379711.patch
Type: text/x-patch
Size: 57407 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211014/fcbaeddd/attachment.bin>


More information about the llvm-commits mailing list