Hi Segher, Thanks for your comments. I checked the cost table. For P9 and P10, the cost of all mul* insn is the same, not relevant to the size of operand.
I will split the test case to one compiling and one runnable case. Thanks. Gui Haochen On 10/8/2022 上午 5:43, Segher Boessenkool wrote: > Hi! > > On Mon, Aug 08, 2022 at 02:04:07PM +0800, HAO CHEN GUI wrote: >> This patch adds an expand and several insns for multiply-add with three >> 64bit operands. > > Also for maddld for 32-bit operands. > >> "maddld %0,%1,%2,%3" >> [(set_attr "type" "mul")]) > > I suppose attr "size" isn't relevant for any of the cpus that implement > these instructions? > > Okay for trunk. Thanks! > > (The testcase improvements can be done later). > > > Segher
