[PATCH] D107058: [SLP]Fix a crash in gathered loads analysis.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 30 03:15:43 PDT 2021


spatel added a comment.

In D107058#2915119 <https://reviews.llvm.org/D107058#2915119>, @ABataev wrote:

> In D107058#2915078 <https://reviews.llvm.org/D107058#2915078>, @spatel wrote:
>
>> Code changes LGTM. No test changes in the updated revision though?
>> Note that there's a new bugzilla report that has another potential test example for this bug:
>> https://llvm.org/PR51275
>
> No test changes and actually there should not be any. It is a corner case.

Ok - I saw the earlier request about adding a comment, and the file name changed, so maybe that’s enough. I didn’t try to reduce the example from bugzilla, but that shows it is possible to induce from legitimate C source code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107058



More information about the llvm-commits mailing list