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