From 9f419393f2a0219c6f2e7ad083e61b370ca36827 Mon Sep 17 00:00:00 2001 From: Eric Botcazou Date: Fri, 1 Feb 2013 10:19:57 +0000 Subject: [PATCH] fold-const.c (make_range_step): Bail out if the range isn't testing for zero. * fold-const.c (make_range_step) : Bail out if the range isn't testing for zero. From-SVN: r195642 --- gcc/ChangeLog | 5 +++++ gcc/fold-const.c | 4 ++++ gcc/testsuite/ChangeLog | 4 ++++ gcc/testsuite/gnat.dg/opt26.adb | 37 +++++++++++++++++++++++++++++++++ 4 files changed, 50 insertions(+) create mode 100644 gcc/testsuite/gnat.dg/opt26.adb diff --git a/gcc/ChangeLog b/gcc/ChangeLog index b8b116a1a0f..aa966f93906 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2013-02-01 Eric Botcazou + + * fold-const.c (make_range_step) : Bail out if the + range isn't testing for zero. + 2013-01-31 Steven Bosscher PR middle-end/56113 diff --git a/gcc/fold-const.c b/gcc/fold-const.c index 855f08fdf75..93f38cbbf21 100644 --- a/gcc/fold-const.c +++ b/gcc/fold-const.c @@ -3813,6 +3813,10 @@ make_range_step (location_t loc, enum tree_code code, tree arg0, tree arg1, switch (code) { case TRUTH_NOT_EXPR: + /* We can only do something if the range is testing for zero. */ + if (low == NULL_TREE || high == NULL_TREE + || ! integer_zerop (low) || ! integer_zerop (high)) + return NULL_TREE; *p_in_p = ! in_p; return arg0; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 867a037b576..151edde20df 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2013-02-01 Eric Botcazou + + * gnat.dg/opt26.adb: New test. + 2013-01-31 Ramana Radhakrishnan Revert. diff --git a/gcc/testsuite/gnat.dg/opt26.adb b/gcc/testsuite/gnat.dg/opt26.adb new file mode 100644 index 00000000000..61f8b5b1eb9 --- /dev/null +++ b/gcc/testsuite/gnat.dg/opt26.adb @@ -0,0 +1,37 @@ +-- { dg-do run } +-- { dg-options "-gnato -O" } + +with Interfaces; use Interfaces; + +procedure Opt26 is + + procedure Shift_Left_Bool + (Bool : in Boolean; + U8 : out Interfaces.Unsigned_8) + is + begin + U8 := Shift_Left (Boolean'Pos (Bool), 6); + end Shift_Left_Bool; + + procedure Shift_Left_Not_Bool + (Bool : in Boolean; + U8 : out Interfaces.Unsigned_8) + is + begin + U8 := Shift_Left (Boolean'Pos (not Bool), 6); + end Shift_Left_Not_Bool; + + Bool : constant Boolean := True; + Byte1, Byte2 : Interfaces.Unsigned_8; + +begin + + Shift_Left_Bool (Bool, Byte1); + + Shift_Left_Not_Bool (Bool, Byte2); + + if Byte1 + Byte2 /= 64 then + raise Program_Error; + end if; + +end; -- 2.30.2