Re: [PATCH, rs6000] Finish implementation of __builtin_atlivec_lvx_v1ti

2018-03-21 Thread Peter Bergner
On 3/14/18 4:27 PM, Kelvin Nilsen wrote: > 2018-03-14 Kelvin Nilsen > > * config/rs6000/rs6000-c.c (altivec_overloaded_builtins): Add > entries for V1TI variants of __builtin_altivec_ld builtin. > * config/rs6000/rs6000.c (altivec_expand_lv_builtin): Add test and > handli

Re: [PATCH, rs6000] Finish implementation of __builtin_atlivec_lvx_v1ti

2018-03-20 Thread Peter Bergner
On 3/14/18 4:27 PM, Kelvin Nilsen wrote: > @@ -14452,6 +14452,7 @@ altivec_expand_lv_builtin (enum insn_code icode, t > LVXL and LVE*X expand to use UNSPECs to hide their special behavior, > so the raw address is fine. */ >if (icode == CODE_FOR_altivec_lvx_v2df_2op > + || icod

Re: [PATCH, rs6000] Finish implementation of __builtin_atlivec_lvx_v1ti

2018-03-16 Thread Segher Boessenkool
On Wed, Mar 14, 2018 at 04:27:31PM -0500, Kelvin Nilsen wrote: > During code review, it was discovered that the implementation of > __builtin_altivec_lvx_v1ti is not complete. The constant > ALTIVEC_BUILTINLVX_V1TI is introduced and is bound to the function > __builtin_altivec_lvx_v1ti. However,

[PATCH, rs6000] Finish implementation of __builtin_atlivec_lvx_v1ti

2018-03-14 Thread Kelvin Nilsen
During code review, it was discovered that the implementation of __builtin_altivec_lvx_v1ti is not complete. The constant ALTIVEC_BUILTINLVX_V1TI is introduced and is bound to the function __builtin_altivec_lvx_v1ti. However, this function's implementation is incomplete because there is no call t