[PATCH] D95631: [CodeGen][AArch64] Add TargetInstrInfo hook to modify the TailDuplicateSize default threshold

[Nicholas Guy via Phabricator via llvm-commits]

NickGuy updated this revision to Diff 321760.
NickGuy marked an inline comment as done.
NickGuy added a comment.

> If this version is working that's great, but check it's still OK.

I haven't seen any issues so far, and in some cases I've seen a slight improvement. I've included the new Spec results, this time there are more significant changes across the suite.

| Benchmark       | TailDuplicator | MachineBlockPlacement |
| --------------- | -------------- | --------------------- |
| 500.perlbench_r | -0.048%        | -0.155%               |
| 502.gcc_r       | 0.036%         | 0.234%                |
| 505.mcf_r       | 0.675%         | 1.349%                |
| 513.xalancbmk_r | -0.240%        | -0.051%               |
| 520.omnetpp_r   | 0.048%         | 0.164%                |
| 525.x264_r      | 0.062%         | 0.185%                |
| 557.xz_r        | 0.417%         | 0.296%                |


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95631

Files:
  llvm/include/llvm/CodeGen/TargetInstrInfo.h
  llvm/lib/CodeGen/MachineBlockPlacement.cpp
  llvm/lib/Target/AArch64/AArch64InstrInfo.cpp
  llvm/lib/Target/AArch64/AArch64InstrInfo.h
  llvm/test/CodeGen/AArch64/aarch64-tail-dup-size.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95631.321760.patch
Type: text/x-patch
Size: 6230 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210205/4632ee23/attachment.bin>