[PATCH] D156879: [AggressiveAntiDepBreaker] Tweak the fix for renaming a subregister of a live register

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 2 05:12:45 PDT 2023


foad created this revision.
foad added reviewers: kparzysz, MatzeB, qcolombet, fhahn, arsenm.
Herald added subscribers: StephenFan, hiraditya.
Herald added a project: All.
foad requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.

This patch tweaks the fix in D20627 <https://reviews.llvm.org/D20627> "Do not rename registers that do not
start an independent live range" to only consider Data dependencies, not
Output or Anti dependencies. An Output or Anti dependency to a superreg
does not imply that that superreg is live at the current instruction.

This enables breaking anti-dependencies in a few more cases as shown by
the lit test updates.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D156879

Files:
  llvm/lib/CodeGen/AggressiveAntiDepBreaker.cpp
  llvm/test/CodeGen/Hexagon/autohvx/fp-to-int.ll
  llvm/test/CodeGen/Hexagon/autohvx/mulh.ll
  llvm/test/CodeGen/Hexagon/autohvx/vmpy-parts.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D156879.546415.patch
Type: text/x-patch
Size: 14747 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230802/312fd2d8/attachment-0001.bin>


More information about the llvm-commits mailing list