[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