https://gcc.gnu.org/bugzilla/show_bug.cgi?id=112943
Hongtao Liu <liuhongt at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |liuhongt at gcc dot gnu.org --- Comment #3 from Hongtao Liu <liuhongt at gcc dot gnu.org> --- (In reply to Jakub Jelinek from comment #1) > Why does ix86_expand_binary_operator have the use_ndd argument at all? > Shouldn't it always act as if the argument is TARGET_APX_NDD? > Or, any particular reason why it isn't done in ashl<mode>3 (but in other > shifts/rotates)? By the time we support apx_ndd, the use_ndd is introduced to enable ndd pattern by pattern so that avoid other patterns crash, and now that we've completed the ndd patch, I think we can try to remove it. We need to make sure that there is no pattern under TARGET_APX_NDD but force a call to ix86_expand_binary_operator with use_ndd as false.