Re: [PATCH v2,rs6000] Add built-in function support for Power9 byte instructions.

2016-12-06 Thread Segher Boessenkool
On Mon, Dec 05, 2016 at 08:56:30PM -0700, Kelvin Nilsen wrote: > +;; Predicate: test byte within range. Just "Test byte within range."? > +;; Return in target register operand 0 a value of 1 if the byte > +;; held in bits 24:31 of operand 1 is within the inclusive range > +;; bounded above by ope

Re: [PATCH v2,rs6000] Add built-in function support for Power9 byte instructions.

2016-12-05 Thread David Edelsohn
Kelvin, This version shows a lot of improvement. (cmprb): New expansion. (*cmprb): New insn. (*setb): New insn. (cmprb2): New expansion. (*cmprb2): New insn. (cmpeqb): New expansion. (*cmpeqb): New insn. The named and un-named patterns should have different names. Technically, the names don't ha