On Mon, Dec 12, 2016 at 06:36:16PM -0700, Martin Sebor wrote:
> +/* Return true if the type of OP is signed, looking through any casts
> +   to an unsigned type.  */
> +
> +static bool
> +operand_signed_p (tree op)
> +{
> +  bitmap visited = NULL;
> +  bool ret = operand_signed_p (op, &visited);
> +
> +  if (visited)
> +    BITMAP_FREE (visited);

I think you can drop the if before BITMAP_FREE here.

        Marek

Reply via email to