Tue, Jun 11
Thu, Jun 6
- Mark ldmxscr as writing to memory. Unfortunately the DAG relies on intrinsics writing to memory to determine side-effet-ness.
- Mark stmxscr as having side effects, as discusse din the review.
- Update tests.
Wed, Jun 5
A side effecting instruction doesn't prevent other non-side effecting instructions from moving across it.
Don’t we need stmxcsr to have side effects to keep it from reordering relative to ldmxcsr?
There are a bunch of tests to update, but I'd like to get opinions on this before going further. I'm not sure why the properties were not already there. Did I miss something ?
Mon, Jun 3
Tue, May 28
Make ExpandMemCmp preserve the DomTree.
Fri, May 24
Thu, May 23
Wed, May 22
add missing inline
Tue, May 21
@courbet Please can you ensure you have EXPENSIVE_CHECKS enabled in all your builds before going any further - you've been breaking the buildbots for well over a week now and you still haven't fixed the underlying issue.
May 17 2019
May 16 2019
Do we have a test for this? I don't think it is a problem but if we do we could update it. Otherwise, LGTM.
May 15 2019
- address comments
- update tests with named blocks.
May 13 2019
May 9 2019
Forgot two test files.
May 6 2019
Fixing RUN lines for PowerPC opt tests.