[PATCH] D88020: [SplitKit] In addDeadDef tolerate parent range that defines more lanes

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 01:48:44 PDT 2020


foad added a comment.

In D88020#2291428 <https://reviews.llvm.org/D88020#2291428>, @kparzysz wrote:

> I don't think it's safe to add dead defs to the extra lanes.  You could split the superset into the matching part and the remaining part, then only operate on the matching one.

I don't understand. We're not adding dead defs for any "extra" lanes. The parent had a subrange with lane mask LF3. The child has two subranges with lane masks LF0 and L03, so together they cover exactly the same lanes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88020



More information about the llvm-commits mailing list