[llvm-commits] [llvm-gcc-4.2] r108206 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp

[Eric Christopher]

On Jul 12, 2010, at 11:42 PM, Duncan Sands wrote:

> Hi Eric,
> 
>> +  /* If this is a vla type and we requested an alignment greater than the stack
>> +     alignment, error out since we're not going to dynamically realign
>> +     variable length array allocations.  We're placing this here instead of
>> +     later in case it's a relatively unused variable.  */
> 
> does gcc support dynamic stack realignment with vla?
> 

Nope.

>> +  if (TREE_CODE (type) == ARRAY_TYPE&&  C_TYPE_VARIABLE_SIZE (type)&&
>> +      DECL_ALIGN(decl)/8u>  TheTarget->getFrameInfo()->getStackAlignment())
> 
> What is C_TYPE_VARIABLE_SIZE?  It sounds like you are calling a C language
> specific function from language independent code.

Somewhat, I changed it later :)

Especially since C++ lowers VLAs by hand.

-eric