On 05/25/2016 03:30 PM, Dominik Vogt wrote:
* explow.c (allocate_dynamic_stack_space): Simplify knowing that
MUST_ALIGN was always true and extra_align ist always BITS_PER_UNIT.
I tried to do some archaeology to find out how the code came to look the
way it currently does. A relevant message appears to be
https://gcc.gnu.org/ml/gcc-patches/2011-01/msg00836.html
There's some discussion about how STACK_POINT_OFFSET shouldn't cause us
to have to align, and postponing that optimization to gcc-4.7. Since
STACK_POINTER_OFFSET should be constant, it ought to be easy enough to
take it into account.
So, I'm undecided. Your cleanup is valid as the code stands right now,
but I'm undecided whether we shouldn't fix the potentially unnecessary
extra alignment instead.
Bernd