On Wed, 7 Feb 2018, Segher Boessenkool wrote: > Hi Mike, > > On Tue, Feb 06, 2018 at 04:34:08PM -0500, Michael Meissner wrote: > > Here is the patch reworked. It bootstraps on both little/big endian power8, > > and all of the tests run. Can I install this into trunk now, and into GCC 7 > > after a soak period (along with the previous patch)? > > > +;; If have ISA 3.0, QI/HImode values can go in both VSX registers and GPR > > "If we have"? > > > + [(set (match_operand:QHSI 0 "memory_operand" "=Z") > > + (any_fix:QHSI (match_operand:SFDF 1 "gpc_reg_operand" "wa"))) > > + (clobber (match_scratch:SI 2 "=wa"))] > > + "((<QHSI:MODE>mode == SImode && TARGET_P8_VECTOR) > > + || (<QHSI:MODE>mode != SImode && TARGET_P9_VECTOR))" > > This is the same as > > "(<QHSI:MODE>mode == SImode && TARGET_P8_VECTOR) || TARGET_P9_VECTOR"
Umm, sorry for chiming in here with zero rs6000 knowledge and I might be missing something trivial but...what wouldn't that misfire for "<QHSI:MODE>mode == SImode && ! TARGET_P8_VECTOR && TARGET_P9_VECTOR" ? (Is that invalid or not applicable or don't care or something?) brgds, H-P
