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]
Other format: [Raw text]

Re: [PATCH] Disable vrp47 for S/390


Hi,

 /* Skip on MIPS, where LOGICAL_OP_NON_SHORT_CIRCUIT inhibits the setcc
    optimizations that expose the VRP opportunity.  */
! /* Skip on S/390.  Lower values in BRANCH_COST lead to two conditional
!    jumps when evaluating an && condition.  VRP is not able to optimize
!    this.  */
! /* { dg-do compile { target { ! mips*-*-* ! s390*-*-* } } } */
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

This isn't correct syntax.

Mmh it worked for me and I hope for mips as well. What is the preferred syntax? There does not seem to be THE way to do it. Searching the testsuite I find various ways:


/* { dg-do compile { target { ! { mips*-*-* s390*-*-* } } } } */
/* { dg-do compile { target { { ! mips*-*-* } && { ! s390*-*-* } } } } */

I have picked the last version since this seems to be the most frequently used one.

Bye,

-Andreas-


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