User Details
- User Since
- Aug 13 2014, 3:59 AM (449 w, 6 d)
Jul 20 2017
Addressed review comments and re-synced with TOT.
Jul 18 2017
Jul 17 2017
Jul 3 2017
LGTM
LGTM
Jun 22 2017
Committed in revision 305682.
Thanks to Simon for the review and for fixing the build breakage.
Committed in revision 305997.
Jun 19 2017
Addressed review comments.
LGTM
Jun 15 2017
Updated diff with latest TOT.
Jun 14 2017
Removed unnecessary addend calculation.
Jun 13 2017
Addressed review comments. Sorry for the delay.
Jun 12 2017
Using preprocessor macro _MIPS_SIM in sanitizer_atomic_clang.h as well.
Addressed review comments
Jun 8 2017
LGTM
May 29 2017
Added 64-bit atomic functions for mips32 which are required in sanitizer common.
May 17 2017
LGTM
May 9 2017
Addressed review comments.
May 4 2017
LGTM
May 3 2017
LGTM
Apr 26 2017
Ping
Committed in revision 301394
Apr 25 2017
LGTM
Addressed review comments
Apr 24 2017
Apr 23 2017
Committed in revision 301158.
Apr 20 2017
Apr 13 2017
Thank you for this. I am curious, what did you test this on?
I have no experience on MIPS, would you be able to provide some help in case of any breakage?
Addressed review comments
Apr 7 2017
Apr 4 2017
LGTM
LGTM
Mar 7 2017
LGTM
Mar 6 2017
Feb 23 2017
LGTM
LGTM with minor nits.
Feb 15 2017
Committed in revision 295166.
Thank you all for the review.
Committed in revision 295164.
Committed in revision 295163
Using llvm-readobj instead of llvm-objdump in test xray-section-group.ll
Adding support for mips64/mips64el.
Feb 13 2017
Addressed review comments.
Addressed review comments.
LGTM with minor nits.
Jan 25 2017
Jan 23 2017
Addressed review comments
Jan 8 2017
Dec 21 2016
Addressed review comments.
Addressed review comments.
Adding mips64 support for xray along with this change since I have the patch ready.
Dec 20 2016
I'm being hit with an llvm assertions when trying to use the integrated assembler about the >function size when not being absolute.
Can you repost the patch with comments addressed, and take a quick look to see if basic sample >programs can be instrumented?
Increased patch context.
Addressed review comments. I've also added mips64 support for xray along with this change since the patch was ready.
Dec 14 2016
Removed patching of instruction "addiu t9, t9, 44" as it was unnecessary since it will always be there at the end of the sled when xray instrumentation is enabled.
Added test for xray attribute support on mips.
Addressed review comments.
Addressed review comments
Dec 13 2016
Added handling of the case where function being instrumented is in a comdat group, as suggested.
Dec 12 2016
Dec 7 2016
Committed in rL289027.
Addressed review comments
Dec 6 2016
Addressed review comments
Dec 2 2016
As suggested, reduced store_context_size to 1 when slow unwinder is being used.
Nov 24 2016
This test is for the ChainedOriginDepotLockAll() logic in msan_interceptors.cc. Could you verify that the fork interceptor is being used on MIPS? I wonder if we need to intercept something else, like vfork.
Nov 23 2016
This test fails on MIPS because many threads are waiting indefinitely on a futex wait. The test pases with reduced number of child processes. Is it okay if we reduce the number of child processes of this test for MIPS?
Nov 11 2016
Committed revision 286583
Nov 6 2016
Removed the restorer field from the kernel sigaction structure for MIPS.
Oct 19 2016
Oct 6 2016
Committed revision 283438
Committed revision 283435