[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