<div style="white-space:pre-wrap">Looks good, thanks for fixing it!<br><br> Filipe</div><br><div class="gmail_quote"><div dir="ltr">On Tue, 8 Nov 2016 at 21:29, Evgeniy Stepanov via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">eugenis accepted this revision.<br class="gmail_msg">
eugenis added a comment.<br class="gmail_msg">
This revision is now accepted and ready to land.<br class="gmail_msg">
<br class="gmail_msg">
LGTM w/ nit<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:641<br class="gmail_msg">
   SmallVector<AllocaInst *, 16> AllocaVec;<br class="gmail_msg">
-  SmallSetVector<AllocaInst *, 16> NonInstrumentedStaticAllocaVec;<br class="gmail_msg">
+  SmallSetVector<AllocaInst *, 16> StaticAllocasToMoveUp;<br class="gmail_msg">
   SmallVector<Instruction *, 8> RetVec;<br class="gmail_msg">
----------------<br class="gmail_msg">
This can be just SmallVector.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26380" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26380</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
_______________________________________________<br class="gmail_msg">
llvm-commits mailing list<br class="gmail_msg">
<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="gmail_msg">
</blockquote></div>