- User Since
- Jul 20 2020, 11:37 AM (37 w, 5 d)
Thu, Apr 1
Mar 9 2021
Rebase to avoid failing tests.
Mar 7 2021
Mar 5 2021
If the rest of the changes are okay then I need someone to commit this for me. Commit can be made with name Mauri Mustonen and email email@example.com, thanks!
Mar 4 2021
Simplify the test case and add more checks to make sure select statement operands and result are used correctly.
Thanks for the review and comments @fhahn. I need someone to submit this patch for me because I don't have commit rights. Commit can be made with name Mauri Mustonen and email of firstname.lastname@example.org, thanks!
Mar 3 2021
Feb 26 2021
Add UseVPlanNative path boolean to inform when to use VPlan native path, removed unnecessary assert modification from earlier and update the test case. Now vectorizing inner loop produces the same vectorized code when VPlan native path flag is enabled and not enabled.
Feb 25 2021
Add test cases for different select conditions and clean up test to only check relevant widened select lines. I checked other tests that other people have written and some of them also check for the relevant lines only, not all lines generated by the python script. So I did the same. Any opinions is this a preferred way?
Add bare minimum test case that I was able to use to reproduce the issues and it passes when the patch is applied. Also add one more assert fix that I noticed came up when testing this.
Feb 24 2021
Feb 23 2021
Removed use of PredicatedScalarEvolution and use ScalaEvolution instead to clean up diff.
Call instruction widening as a separate patch here: https://reviews.llvm.org/D97278
Rebase with latest changes, add support for select instruction widening only in this patch and add test case for it. Add call instruction widening as a separate patch later.
Feb 21 2021
Fix failing unit tests.
Accidentally duplicated this with arcanist. Original diff here: https://reviews.llvm.org/D97136
Add support to widen call and select instructions in VPlan native path by using correct recipes when such instruction is encountered. This is already used by inner loop vectorizer. I modified my previously committed VPlan test to also test for the call instruction widening.
Feb 20 2021
Oct 6 2020
Oct 5 2020
Ping! Does someone want to commit this for me with the name Mauri Mustonen and email email@example.com, thanks!
Sep 19 2020
Squashed diff. And I need someone to commit this change for me. It can be done with the name Mauri Mustonen and email firstname.lastname@example.org, thanks.
Sep 18 2020
Typed all used LLVM intrinsic functions.
Sep 17 2020
Updated FileCheck to check more detailed about vectorized basic blocks. But I'd still want to get some pointers. Is this now too detailed checking and making the check too brittle for changes?
Sep 12 2020
Aug 17 2020
I need someone else to commit this patch for me. Commit can be made with name Mauri Mustonen and email email@example.com.