[PATCH] D26380: [asan] Speed up compilation of large C++ stringmaps (tons of allocas) with ASan

Filipe Cabecinhas via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 8 13:32:33 PST 2016


Looks good, thanks for fixing it!

Filipe

On Tue, 8 Nov 2016 at 21:29, Evgeniy Stepanov via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> eugenis accepted this revision.
> eugenis added a comment.
> This revision is now accepted and ready to land.
>
> LGTM w/ nit
>
>
>
> ================
> Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:641
>    SmallVector<AllocaInst *, 16> AllocaVec;
> -  SmallSetVector<AllocaInst *, 16> NonInstrumentedStaticAllocaVec;
> +  SmallSetVector<AllocaInst *, 16> StaticAllocasToMoveUp;
>    SmallVector<Instruction *, 8> RetVec;
> ----------------
> This can be just SmallVector.
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D26380
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161108/a0ab2d5f/attachment.html>


More information about the llvm-commits mailing list