On Wed, Jul 26, 2023 at 01:54:01PM +0800, Kewen.Lin wrote: > Hi Mike, > > on 2023/7/11 03:59, Michael Meissner wrote: > > In doing other work, I noticed that there was an insn: > > > > vsx_extract_v4sf_<mode>_load > > > > Which did not have an iterator. I removed the useless <mode>. > > It actually has a mode iterator, the "P" is used for clobber. > > The whole pattern of this define_insn_and_split is > > (define_insn_and_split "*vsx_extract_v4sf_<mode>_load" > [(set (match_operand:SF 0 "register_operand" "=f,v,v,?r") > (vec_select:SF > (match_operand:V4SF 1 "memory_operand" "m,Z,m,m") > (parallel [(match_operand:QI 2 "const_0_to_3_operand" "n,n,n,n")]))) > (clobber (match_scratch:P 3 "=&b,&b,&b,&b"))] <== *P used here* > > Its definition is: > > (define_mode_iterator P [(SI "TARGET_32BIT") (DI "TARGET_64BIT")]) > > I guess we can just leave it there? > > BR, > Kewen
Yes, I didn't notice the :P in the insn. -- Michael Meissner, IBM PO Box 98, Ayer, Massachusetts, USA, 01432 email: meiss...@linux.ibm.com
