[PATCH] D23799: [ESan][MIPS] Adds support for MIPS64
Derek Bruening via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 26 16:05:11 PDT 2016
bruening added inline comments.
================
Comment at: lib/esan/esan.cpp:150
@@ +149,3 @@
+ // This is based on the assumption that the intial stack is always allocated
+ // in the topmost segment of the virtual address space and the assumption
+ // holds true on all the platforms currently supported.
----------------
nit: for x86-64 the vsyscall region is a higher address than the stack
================
Comment at: lib/esan/esan_shadow.h:136
@@ -135,3 @@
-// We'll want to use templatized functions over the ShadowMapping once
-// we support more platforms.
-#error Platform not supported
----------------
This was overlooked?
Repository:
rL LLVM
https://reviews.llvm.org/D23799
More information about the llvm-commits
mailing list