Re: [PATCH], Simplify setup of complex types

[Jeff Law <law@redhat.com>]

On 06/21/2016 07:48 AM, Michael Meissner wrote:
> When I submitted the back port to allow complex __float128 to be created on the
> PowerPC to the GCC 6.2 branch, Richard Biener suggested a simpler way to set
> the complex type:
> https://gcc.gnu.org/ml/gcc-patches/2016-06/msg01114.html
>
> This patch implements this change for the trunk.  I have a companion patch for
> 6.2 once this goes into the trunk.
>
> I bootstrapped the compiler and did a make check with no regressions on a big
> endian Power 7 system and a little endian Power 8 system.  Is it ok to go into
> the trunk?
>
> [gcc]
> 2016-06-21  Michael Meissner  <meissner@linux.vnet.ibm.com>
>
> 	* stor-layout.c (layout_type): Move setting complex MODE to
> 	layout_type, instead of setting it ahead of time by the caller.
> 	* tree.c (build_complex_type): Likewise.
>
> [gcc/fortran]
> 2016-06-21  Michael Meissner  <meissner@linux.vnet.ibm.com>
>
> 	* trans-types.c (gfc_build_complex_type): Move setting complex
> 	MODE to layout_type, instead of setting it ahead of time by the
> 	caller.
OK.

Jeff

ps.  Thanks for the pointer back to the prior discussion with Richi -- 
that makes it a lot easier to pick up state on this while Richi is on PTO.