This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: A new patch for floating point conditional mov on PPo


1) What prevents gcc from accidently matching the fcmov test / compare
   insns by adding a clobber to an existing compare insn?  There's a note
   in md.texi about the combiner adding / deleting clobber expressions
   as necessary.

2) In output_int_test_compare_fcmov you have:

     if (CONSTANT_P (operands[0]) || GET_CODE (operands[1]) == MEM)
       output = reverse;
     else
       output = normal;

  a) I believe that the patterns require operand 0 to be a
     nonimmediate_operand in which case how can CONSTANT_P
     ever be true?

  b) I believe that Intel allows operand 1 to be a memory
     operand in which case why does anything special need
     to be done?

-- John
-------------------------------------------------------------------------
|   Feith Systems  |   Voice: 1-215-646-8000  |  Email: john@feith.com  |
|    John Wehle    |     Fax: 1-215-540-5495  |                         |
-------------------------------------------------------------------------



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]