[PATCH] D49942: [ASAN] Port asan to FreeBSD/mips64.

John Baldwin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 13:25:55 PDT 2018


bsdjhb added inline comments.


================
Comment at: lib/asan/asan_mapping.h:220
 #    define SHADOW_OFFSET kSystemZ_ShadowOffset64
-#  elif SANITIZER_FREEBSD
+#  elif SANITIZER_FREEBSD && !defined(__mips64)
 #    define SHADOW_OFFSET kFreeBSD_ShadowOffset64
----------------
jrtc27 wrote:
> Is there a reason why you don't just hoist the `__mips64` case up to line 215 to come before this, like is done for 32-bit MIPS on line 188?
Actually, I meant to ask that as a comment when I posted this in terms of what do folks think is the better approach.  Thanks for the reminder.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D49942





More information about the llvm-commits mailing list