[PATCH 2/7] Reset the range info on the moved instruction in PHIOPT

Jeff Law jeffreyalaw@gmail.com
Thu Jun 24 15:11:48 GMT 2021



On 6/23/2021 4:19 PM, apinski--- via Gcc-patches wrote:
> From: Andrew Pinski <apinski@marvell.com>
>
> I had missed this when wrote the patch which allowed the
> gimple to be moved from inside the conditional as it.  It
> was also missed in the review.  Anyways the range information
> needs to be reset for the moved gimple as it was under a
> conditional and the flow has changed to be unconditional.
> I have not seen any testcase in the wild that produces wrong code
> yet which is why there is no testcase but this is similar to what
> the other code in phiopt does so after moving those to match, there
> might be some.
>
> OK? Bootstrapped and tested on x86_64-linux-gnu with no regressions.
>
> gcc/ChangeLog:
>
> 	* tree-ssa-phiopt.c (match_simplify_replacement): Reset
> 	flow senatitive info on the moved ssa set.
OK
jeff



More information about the Gcc-patches mailing list