[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