[PATCH, rs6000] gimple folding support for vec_pack and vec_unpack

Segher Boessenkool segher@kernel.crashing.org
Wed Jul 11 22:53:00 GMT 2018


Hi!

On Mon, Jul 09, 2018 at 02:08:37PM -0500, Will Schmidt wrote:
> 	* config/rs6000/rs6000.c (rs6000_gimple_fold_builtin):
> 	Add support for gimple-folding of vec_pack() and vec_unpack()
> 	intrinsics.

> +    case ALTIVEC_BUILTIN_VUPKHPX:
> +    case ALTIVEC_BUILTIN_VUPKLPX:
> +      {
> +       return false;
> +      }

A block around a signle statement looks a bit silly (and in the other
cases in your patch it isn't necessary either; it is nice if you use it
to give some scope to a local var, but you don't have that here).

But, patch is fine as far as I can see :-)


Segher



More information about the Gcc-patches mailing list