[PATCH] D23799: [ESan][MIPS] Adds support for MIPS64

Sagar Thakur via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 06:41:21 PDT 2016


slthakur added inline comments.

================
Comment at: lib/esan/esan_shadow.h:127
@@ +126,3 @@
+  {0xaa00000000u, 0xab00000000u, false},
+  {0xff00000000u, 0xffffffffffu, true},
+};
----------------
bruening wrote:
> The final 2 aren't adjacent, so I don't understand why they would be merged?
Yes, the last two don't need to be merged.

================
Comment at: lib/sanitizer_common/sanitizer_linux_mips64.S:20
@@ +19,3 @@
+        syscall
+        nop
+
----------------
bruening wrote:
> Why a nop?
Sorry a nop is not required there. I felt that a syscall would require a delay slot but thats not the case.


Repository:
  rL LLVM

https://reviews.llvm.org/D23799





More information about the llvm-commits mailing list