- User Since
- Jun 14 2019, 2:23 AM (104 w, 1 d)
Wed, Jun 9
Rebasing, no new test changes.
Fri, Jun 4
- More affected tests:
Wed, Jun 2
Sat, May 29
- Add an example of the new copy propagation in the top comments
- Style fixes
Apr 18 2021
Apr 15 2021
Apr 5 2021
Add tests suggested by @jmorse
Apr 2 2021
Mar 27 2021
Uh, I just figured out how mir tests work. Guess nobody wanted to point out how dumb I am. 😅
Mar 24 2021
Thanks @jmorse for the review.
Mar 18 2021
Mar 17 2021
Since this patch already changed output of many test cases, do we still need a test case specifically for this patch?
Update Mips tests.
Include full context in the patch
Mar 16 2021
I reviewed the changes in CodeGen tests to the best of my ability, and they look fine to me. If someone else could review and confirm, that would be appreciated.
Strengthen the invariant maintained by the CopyTracker.
Mar 15 2021
https://reviews.llvm.org/D98659 could supersede this
Adjust a debug message.
Remove a TODO which is no longer relevant
Adjust how DBG_VALUE is rewritten.
Fix typos in comments.
Things still need to be done:
Mar 12 2021
I think it might be valuable to extend this to all cases like this:
Mar 11 2021
Because you replaced $rbp with $rbx, surely you would have to replace it everywhere. Removing the DBG_VALUE is potential information loss.
renamable $rbx = nuw LEA64r %stack.5.agg.tmp.i, 1, $noreg, 8, $noreg DBG_VALUE $rbp, $noreg, !"this", !DIExpression(), debug-location !293; hot.cpp:0 @[ hot.cpp:84:39 @[ hot.cpp:108:11 ] ] line no:0