On Wed, Aug 24, 2011 at 10:47 AM, Tom de Vries <vr...@codesourcery.com> wrote: > Hi Richard, > > On 07/30/2011 09:21 AM, Tom de Vries wrote: >> This is an updated version of the patch. I have 2 new patches and an updated >> testcase which I will sent out individually. >> >> Patch set was bootstrapped and reg-tested on x86_64. >> >> Ok for trunk? >> > > You already approved the the 2 new patches, and I checked them in. > > Ping for the updated patch and updated testcase: > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02730.html > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02733.html > > The patch replaces a vla __builtin_alloca that has a constant argument with an > array declaration.
Ok with + /* Declare array. */ + elem_type = build_nonstandard_integer_type (BITS_PER_UNIT, 1); + n_elem = size * 8 / BITS_PER_UNIT; + align = MIN (size * 8, GET_MODE_PRECISION (word_mode)); changed to use BIGGEST_ALIGNMENT instead of GET_MODE_PRECISION (word_mode). For the future, can you please include the testcases in the main patch instead of sending them separately? You're the only one separating them and it's really not helpful. Thanks, Richard. > Thanks, > - Tom >
