- User Since
- Jan 23 2018, 4:17 PM (103 w, 3 d)
@Ayal , would be nice if you find time to look at this patch since next two (accepted) patched depends on this one.
Thu, Jan 16
Moved regression test to a separate review.
Wed, Jan 15
Thu, Jan 9
Introduce control flag for the feature and disable by default + Rebase.
ebrevnov committed rGf0abe820eebf: [LoopUtils][NFC] Minor refactoring in getLoopEstimatedTripCount. (authored by ebrevnov).
Fixed raised issue + Rebase.
Wed, Jan 8
Mon, Dec 30
Factor out NFC stuff + rebase
One more round of updates.
ebrevnov committed rG948e745270de: [LV][NFC] Keep dominator tree up to date during vectorization. (authored by ebrevnov).
ebrevnov committed rG1b6286b945a5: [LV][NFC] Some refactoring and renaming to facilitate next change. (authored by ebrevnov).
Thu, Dec 26
Hi @Ayal. Thanks for you input. I fixed all places as you suggested. Please check.
Updated as requested.
Wed, Dec 25
Updated according to comments.
Sun, Dec 22
Fri, Dec 20
Dec 18 2019
Dec 17 2019
Dec 16 2019
Dec 11 2019
Yes there are a dependencies between change sets. Especially this and next one. The order is defined by parent/child relationship and can be found at "Revision Contents" section Stack tab.
Dec 10 2019
These extra branches are very short leaved and optimized out by SimplifyCFG which happens just 3 passes after the vectorizer. Thus I don't believe it can cause any harm to the performance. I double check that LLVM's test-suite has no regressions.
Dec 9 2019
Updating as requested by Florian.
Addressing issues raised by Ayal.
Dec 6 2019
Minor naming update
Use SplitBlock utility to automatically udpate DT and LI.
Minor formatting update.
Dec 5 2019
Yes, this case came from the real world benchmark. This change gives +15% on it.
Restore lost change.
Extracted formatting and renaming to a separate review.