- User Since
- Apr 29 2020, 8:06 AM (59 w, 1 d)
May 8 2021
Rebased onto upstream/main
May 7 2021
Forgot to rebase onto upstream... That explains why it went so smoothly!
Rebasing to master
Cool :) This is my first submission, so I don't have commit access. If I understand correctly that means I need someone else to commit it on my behalf, yes?
Apr 23 2021
Fixed formatting issue
Apr 22 2021
Split CHECKs to keep them closer to their relevant functions, and added debug information to disambiguate instructions where relevant.
Apr 20 2021
It should be ready to review again now :)
Removed undefs in test cases.
After a long wait I'm now finally able to work on this again! Please excuse the delay.
Addressed review comments and fixed a crash for non-local variables.
Aug 14 2020
May 7 2020
Updating D79564: [Loop-Vectorize] update test cases to accept new ouput
May 2 2020
Updating D79097: reformatting code according to pre-merge check
Accidentally created an extra revision
Updating D79097: [GVN] Improve analysis for missed optimization remark
[GVN] [Remarks] Trying to push missing file
This is my first time using both Phabricator and Arcanist, so I'm not entirely familiar with the UI and workflow, but I can't seem to find one of the test cases I added in my update. It's supposed to be in llvm/test/Transforms/GVN/opt-remarks-multiple-users.ll, but I can't see it added in the revision. Did I miss something? It's an earlier commit than the new diff I pushed, so I would've thought it would be included as well.
[GVN] Update analysis for optimization remark and add tests