[PATCH] D37611: [IfConversion] More simple, correct dead/kill liveness handling

[Jesper Antonsson via Phabricator via llvm-commits]

JesperAntonsson updated this revision to Diff 114596.
JesperAntonsson added a comment.

Thanks for the reviews, everybody! Here I'm submitting an updated patch based on Matthias' suggestions.

Hope I covered everything, shout if not. (The range-based method extraction might be a good idea, but I'm leaving that to anyone that has use for it.)

As far as I have understood, DebugValues are not allowed in bundles, but I moved the check to the operand anyway. But the question made me think about bundling... Do I need to go inside bundles to adjust as well? I'm only doing bundle heads here.

Since I'm now adding flags as well, I had to update a few tests more.


https://reviews.llvm.org/D37611

Files:
  include/llvm/CodeGen/LivePhysRegs.h
  lib/CodeGen/IfConversion.cpp
  lib/CodeGen/LivePhysRegs.cpp
  test/CodeGen/Hexagon/branch-folder-hoist-kills.mir
  test/CodeGen/Hexagon/ifcvt-live-subreg.mir
  test/CodeGen/Hexagon/livephysregs-add-pristines.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37611.114596.patch
Type: text/x-patch
Size: 8379 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170911/3752dc4f/attachment.bin>