[PATCH] D23363: [compiler-rt] Add support for the dynamic shadow allocation

Etienne Bergeron via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 10 11:58:32 PDT 2016


etienneb added a comment.

In https://reviews.llvm.org/D23363#511583, @kcc wrote:

> I am not a fan of this change because it adds extra complexity and makes asan slower.
>  As I've commented in the other patch, I first want to know for sure that having a fixed address is impossible. 
>  So far I have not seen an explanation.


This is not making ASAN slower, except for the cases where it's configured to use a dynamic shadow.
So, Asan on every plateform should be the same.... except on win64.


https://reviews.llvm.org/D23363





More information about the llvm-commits mailing list