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.

Reply via email to