On 02/09/11 16:01, Richard Sandiford wrote:
> CC_NCV rightly only allows GE(U) and LT(U). GT(U) and LE(U) have to
> implemented by reversing the condition. This is handled correctly when
> the condition is first expanded, but nothing stops later optimisers from
> producing invalid forms.
>
> Thi
>
> My patch has now survived a Thumb-2 bootstrap, and it sounds like Ramana
> has also successfully bootstrapped your original patch.
I'd rather take the version that handles the cases for the dominance
modes as well. RichardE, do you think you could have a look at this
one ?
cheers
Ramana
Chung-Lin Tang writes:
> Hi Richard, this looks very similar to this patch, originally for LP:689887:
> http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00794.html
> Apart from your additional handling in the dominance modes cases.
Indeed. Sorry about that. It must look odd that I've posted such
a
Hi Richard, this looks very similar to this patch, originally for LP:689887:
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00794.html
Apart from your additional handling in the dominance modes cases.
I remember that last patch was held down because Thumb-2 native
bootstrap failed. Did you try that
CC_NCV rightly only allows GE(U) and LT(U). GT(U) and LE(U) have to
implemented by reversing the condition. This is handled correctly when
the condition is first expanded, but nothing stops later optimisers from
producing invalid forms.
This patch makes arm_comparison_operator check that the con
