[PATCH] D91781: [VE] Add regression test for D91151
Kazushi Marukawa via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 20 04:53:39 PST 2020
kaz7 added a comment.
In D91781#2407138 <https://reviews.llvm.org/D91781#2407138>, @dblaikie wrote:
> With the use of CReduce <https://embed.cs.utah.edu/creduce/> & some manual reduction, I managed to get the test down to this:
Thank you so much. I wanted simplify this test like your example. I really appreciate that you show it. I also understand that I definitely need to try creduce and other tools.
> Though it'd be good if you could check this over and ensure this test makes sense for the failure - and see if there's any domain-specific knowledge you could apply to change if it could be modified/simplified/be more explicit about how/why this code is what's needed to reach the buggy case.
I will do that. Your test case causes the identical problem. I can inspect it now since it is much simplified. Thanks so mcuh.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91781/new/
https://reviews.llvm.org/D91781
More information about the llvm-commits
mailing list