[PATCH] D136675: [MachineCSE] Allow PRE of instructions that read physical registers

John Brawn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 08:31:11 PDT 2022


john.brawn updated this revision to Diff 472023.
john.brawn added a comment.

Adjusted to find physical register uses and defs by iterating through all operands, as the defs list doesn't include implicit defs. Added a test based on the code in ffmpeg that was getting incorrectly optimised.


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

https://reviews.llvm.org/D136675

Files:
  llvm/lib/CodeGen/MachineCSE.cpp
  llvm/test/CodeGen/AMDGPU/GlobalISel/hip.extern.shared.array.ll
  llvm/test/CodeGen/AMDGPU/GlobalISel/udiv.i64.ll
  llvm/test/CodeGen/AMDGPU/GlobalISel/urem.i64.ll
  llvm/test/CodeGen/AMDGPU/selectcc-opt.ll
  llvm/test/CodeGen/ARM/machine-cse-cmp.ll
  llvm/test/CodeGen/PowerPC/machine-cse-rm-pre.mir
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/memcall.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136675.472023.patch
Type: text/x-patch
Size: 35291 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221031/d773c7d3/attachment.bin>


More information about the llvm-commits mailing list