[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