[PATCH] D102476: [VectorCombine] Use constant range info for index scalarization legality.

Nuno Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 25 04:01:37 PDT 2021


nlopes accepted this revision.
nlopes added a comment.
This revision is now accepted and ready to land.

In D102476#2779213 <https://reviews.llvm.org/D102476#2779213>, @fhahn wrote:

> In D102476#2777021 <https://reviews.llvm.org/D102476#2777021>, @nlopes wrote:
>
>> This patch looks good if we want to go with the poison-on-OOB semantics.
>> I think it's important to settle that first, please see my bug report: https://bugs.llvm.org/show_bug.cgi?id=50382
>
> I think the conclusion of the discussion at the patch to far seems to be to keep it poison. So I think we could move ahead with this patch?

SGTM!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102476



More information about the llvm-commits mailing list