[cfe-commits] r84981 - /cfe/trunk/lib/AST/ASTContext.cpp

Fariborz Jahanian fjahanian at apple.com
Sat Oct 24 11:03:47 PDT 2009


On Oct 23, 2009, at 6:03 PM, Ted Kremenek wrote:

> Hi Fariborz,
>
> It's great that you did this, but please don't use alloca.  It's not  
> portable, and is strictly prohibited in the LLVM codebase.  Please  
> use a SmallVector or something else.

It is apparently portable enough to be used extensively in gcc front- 
end. But I agree that in clang land we should have a container-based  
solution.
  I did this on top of what was already there. Long term (soon), I  
will convert this unless Mike beat me to it.

- Fariborz





More information about the cfe-commits mailing list