On Wed, Nov 30, 2016 at 08:35:08AM -0700, Kelvin Nilsen wrote: > This patch corrects an error in a patch committed on 2016-10-18 to add > built-in function support for Power9 string operations. In that > original patch, the mode for operand 2 of the newly added vector > extract half-word and full-word instruction patterns was described as > V16QI, even though those instruction patterns were conceptually > operating on V8HI and V4SI operands respectively. > > This patch changes the modes of the operands for these instruction > patterns to better represent the intended types. This patch improves > readability and maintainability of code. It does not affect > correctness of generated code, since the existing implementation > implicitly coerces the operand types to the declared type. > > The patch has been bootstrapped and tested on powerpc64le-unknown-linux > without regressions. > > Is this ok for the trunk?
Okay. Thanks, Segher > 2016-11-30 Kelvin Nilsen <kel...@gcc.gnu.org> > > PR target/78577 > * config/rs6000/vsx.md (vextuhlx): Revise mode of operand 2. > (vextuhrx): Likewise. > (vextuwlx): Likewise. > (vextuwrx): Likewise.
