[PATCH] reorganize simplify_const_relational_operation

Eric Botcazou ebotcazou@libertysurf.fr
Tue Sep 18 16:58:00 GMT 2007


> But this time I had checked right :-) -- it is not this FMT_*, but the
> one just below:
>
>    /* This format converts arguments of types determined by the
>       format string.  */
>    FMT_FLAG_ARG_CONVERT = 1,
>    /* The scanf allocation 'a' kludge applies to this format kind.  */
>    FMT_FLAG_SCANF_A_KLUDGE = 2,
>    /* A % during parsing a specifier is allowed to be a modified % rather
>       that indicating the format is broken and we are out-of-sync.  */
>    FMT_FLAG_FANCY_PERCENT_OK = 4,

OK.  I'll note that there is no enumeration type though, so you don't have

static rtx
comparison_result (enum rtx_code code, enum cr_flags known_results)

         int cr;
          cr = (h0s < h1s || (h0s == h1s && l0u < l1u)) ? CR_LT : CR_GT;
          cr |= (h0u < h1u || (h0u == h1u && l0u < l1u)) ? CR_LTU : CR_GTU;
          return comparison_result (code, cr);


Btw, is this type frobbing OK in C++?

-- 
Eric Botcazou



More information about the Gcc-patches mailing list