------- Comment #9 from krebbel at gcc dot gnu dot org  2005-12-07 17:33 -------
(In reply to comment #8)
> Ok (not sure if it really is a good idea to make the *_operand names that
> long), 
Mmmh you are right but I couldn't think of a better name that moment.

just I'm afraid ashrdi3_cc_64_and, ashrdi3_cc_31_and and ashrsi3_cc_and
> will never match in your patch, because you haven't replaced (match_dup 2)
> with (and:SI (match_dup 2) (match_dup 3)).
Oops. Right - thanks for fixing this.

> Attaching updated version of your patch with this and testcase incorporated.
Thanks I've posted your updated version on gcc-patches and added your name to
the changelog entry.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25268

Reply via email to