- User Since
- Nov 23 2015, 1:58 PM (177 w, 5 d)
Thu, Apr 18
Tue, Apr 2
It's not sufficient to check if you can merge two stores into a valid node; there are backends where you need 4 or more to get a legal merged store.
Fri, Mar 29
rL357283 supersedes this.
Thu, Mar 28
Consider inserted nodes for pruning.
Defer Worklist modification in NodeInserter to follow up patch (D58070).
Now that lifetime node hashing and TokenFactor cleanup has happened, factor out Dead Store improvements as they are no longer needed to prevent regressions.
Wed, Mar 27
Debug test also changes. h
Initial commit reverted due to Halide compile time explosion.
Address lingering test comments.
Tue, Mar 26
Landed in rL356996
Extract copied code into helper and format
Revert mips test changes
This seems like it a natural thing to do as with the X86 backend, but since it doesn't make any test change differences, perhap it is unecessary.
Summarizing offline discussion with jyknight for future reference:
Rebase to tip before landing.
Mon, Mar 25
Address James' Comments.
Check HasSideEffects in analysis for inlineasm.
Mar 18 2019
Address James' comments.
It fell off the end of my work queue, but yes.
Copy over Simon's suggestion.
Mar 15 2019
Realized explanatory comment and associate fixup had been lost in patch stack. Folding it back in.
Mar 13 2019
Mar 12 2019
Mar 8 2019
Abandoning this in favor of D59039
Mar 6 2019
This is a refactored and rebased version of D49691 so that it looks reasonable on Phabricator. I've factored out a initial NFC patch (the first snapshot in history) to highlight the actual functional change.
patch on to p of refactoring.
NFC Refactoring of GatherAllAliases on top of which the patch will occur.
Mar 4 2019
Feb 28 2019
Feb 27 2019
Rebase to tip and add FIXME for regression.
Rebase after some cleanups. Add comments on regressions test cases. Most of these are exposing PR40881. '
The remain case (vsel-cmp-load.ll) appears to be a missing revisitation some nodes
Feb 26 2019
Feb 25 2019
Convert to bit-level sizing for contains. I'm leaving equalBaseIndex as byte-level for now, but a followup NFC patch is probably warranted.