[RFC][IPA-VRP] Re-factor tree-vrp to factor out common code

kugan kugan.vivekanandarajah@linaro.org
Tue Aug 16 08:09:00 GMT 2016




On 23/07/16 20:12, kugan wrote:
> Hi Richard,
>
>>> As we had value_range_type in tree-ssanames.h why not put value_range there?
>>>
>> For IPA_VRP, we now need value_range used in ipa-prop.h (in ipa-vrp
>> patch). Based on this, attached patch now adds struct value_range to
>> tree-ssanames.h and fixes the header files. Please note that I also had
>> to add other headers in few places due to the dependency. Are you OK
>> with this ?
> Here is alternate patch where we keep struct value_range and enum
> value_range_type to tree-vrp.h. May be it is a better approach? Please
> let me know what is your preference.
>

Ping?

This patch places value_range_type and value_range in tree-vrp.h. May be 
this is better?

Alternate patch which keeps value_range_type and value_range in 
tree-ssanames.h is in:
https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01491.html

I also added the necessary header files changed needed for ipa-vrp as 
part of this patch so that changes needed are clear.

Thaks,
Kugan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Factor-out-and-expose-value_range-api-for-IPA-VRP.patch
Type: text/x-patch
Size: 16495 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20160816/e59db4c3/attachment.bin>


More information about the Gcc-patches mailing list