User Details
- User Since
- Jan 20 2014, 12:15 AM (487 w, 5 d)
Mar 18 2015
Is there anything else that needs to be done for this patch?
Feb 27 2015
LGTM
Feb 25 2015
LGTM
Feb 19 2015
LGTM
Feb 18 2015
LGTM
Feb 12 2015
Removed the unnecessary cast.
Feb 6 2015
Rebased the patch. Created new method MipsInstrInfo::getDebugLoc().
Feb 4 2015
LGTM, with additional comment.
Jan 28 2015
LGTM
Jan 27 2015
Jan 26 2015
LGTM
Jan 22 2015
Jan 20 2015
LGTM, with a change.
LGTM
Jan 13 2015
LGTM
Jan 12 2015
LGTM
Dec 30 2014
LGTM, with a change.
Dec 26 2014
Dec 23 2014
LGTM
LGTM
Please include disassembler tests in the patch.
This patch doesn't correctly handle the case when (offset % 4) != 0. For example, for the following instruction
Dec 22 2014
Is there a MIPS breakpoint/trap instruction that would work here?
Dec 18 2014
Yes. Currently two tests fail because of the redundant sandboxing - syscall_return_sandboxing_test and exception_test. For syscall_return_sandboxing_test, to make the test pass we can either remove sandboxing from the assembly, or apply this patch. For exception test, neither removing the sandboxing nor applying the patch will work. Here is the problematic inline assembly from exception_test.c file:
LGTM
Dec 16 2014
LGTM
Dec 11 2014
LGTM
Nov 28 2014
LGTM
Nov 27 2014
LGTM, with two changes.
LGTM, with two changes.
LGTM
Nov 26 2014
LGTM
Nov 25 2014
Nov 24 2014
LGTM
LGTM, with 2 changes.
LGTM, with 3 changes.
Nov 21 2014
LGTM, with two changes.
Nov 18 2014
For the following illegal instructions the error is not reported:
LGTM
Nov 17 2014
Nov 14 2014
Nov 13 2014
LGTM
Nov 12 2014
LGTM
LGTM, with two changes.
LGTM
LGTM. You may optionally add a comment in the test file that this test checks that encoding for srl is correct when fast-isel for mips32 is used.
Nov 10 2014
LGTM
Nov 6 2014
Closed by commit rL221450 (authored by @sstankovic).
Nov 5 2014
LGTM, with two changes.
Nov 4 2014
Implemented in D4903.