[PATCH] D32895: [ASAN] Insert call to __asan_init and load of dynamic shadow address in correct order

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 11:23:20 PDT 2017


kcc added a comment.

Ok.. makes sense. These did not collide before because maybeInsertDynamicShadowAtFunctionEntry is mostly used on WIndows and maybeInsertAsanInitAtFunctionEntry only on Mac.  grrr.
Now it does make sense, but I would prefer if the code did not have a duplicated call to maybeInsertAsanInitAtFunctionEntry
(and, of course, a test)


Repository:
  rL LLVM

https://reviews.llvm.org/D32895





More information about the llvm-commits mailing list