On Tue, Jul 27, 2021 at 10:29:13PM -0400, David Edelsohn via Gcc-patches wrote: > > Add a naive implementation of the subject x86 intrinsic to > > ease porting. > > > > 2021-07-15 Paul A. Clarke <p...@us.ibm.com> > > > > gcc > > * config/rs6000/smmintrin.h (_mm_minpos_epu16): New. > > Segher already approved this with the changes requested.
Segher said: | This does not compute the index correctly for big endian (it needs to | walk from right to left for that). The construction of the return value | looks wrong as well. | | Okay for trunk with that fixed. Thanks! I responded: | I'm not seeing the issue here. The values are numbered by element order, | and the results are in the "first" (minimum value) and "second" (index of | first encountered minimum value in element order) elements of the result. I did not get a response, nor did I change any code. It feels like a stretch to equate the above exchange to "approved", so I'll continue to wait for explicit approval. PC