On 12/13/2016 03:52 AM, Marek Polacek wrote:
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.
Correct. jeff
