May 6 2019
May 5 2019
Apr 30 2019
Apr 25 2019
Apr 23 2019
- Move all tests out of CodeGen.
- Update PowerPC tests.
- Add EarlyCSE to addMemcmpPasses().
Apr 17 2019
As discussed offline, I think this should go through an RFC process.
Apr 11 2019
There is still some test fixing to do for Power, but before I do that I'd like to get your opinion on the approach, in particular regarding the pass placement (I pretty much placed it randomly here).
[ExpandMemCmp][MergeICmps] Move passes out of CodeGen into opt pipeline.
Apr 10 2019
Alternatively, what do you think about making ExpandMemCmp a late IR optimization pass like the vectorizer passes?
Apr 5 2019
Apr 3 2019
Are you sure ?
Apr 1 2019
Thanks Roman. I only have a comment regarding documentation.
I'll let gchatelet@ review that one as he started looking at doing that some time ago.
OK, I succeeded in profiling this using pprof instead of callgrind.
Could that be the reason for the regression ?
This made pdfium use 6.8% more cpu, https://bugs.chromium.org/p/chromium/issues/detail?id=947611
Since the intent here was to make things faster, any ideas how this could happen?
Mar 29 2019
Submitted as 357239.
Thanks, just one nit.
Mar 28 2019
What about CPU's that specify let PostRAScheduler = 1; ?
This could use test coverage i guess?
I don't think is a blocking issue, but in future we should revisit the logic in X86MacroFusion.
Only cosmetic comments, this looks good !
Thanks for the review.
Address review comments, rebase.
Mar 27 2019
Update comment, swap features.
OK cool thanks for having a look.
Mar 26 2019
Add unit test.
The issue is that such a test would have failed in a non-deterministic manner, which is why I've only added a unit test. But now that this should not longer fail, let's add one.
To reword: because if i do simple clustering by opcode, i will then need to add yet another
"stabilization" step - for each cluster, check that every measurement is neighbor of all
the other points in that cluster, and if they are not, mark cluster as noise.
(well, not every vs. every, just the lower/upper triangle excluding diagonal)
Mar 25 2019
Mar 22 2019
Ignore template contexts and add a test.
I plan to run some experiments today using your patch.