Re: [PATCH 4/4] Fix scaling of a sreal number.

2018-09-24 Thread Jeff Law
On 9/23/18 1:08 AM, marxin wrote: > > The patch is addressing following LLVM warning: > gcc/ipa-fnsummary.c:2745:54:Value Conversion Issue: implicit conversion from > 'double' to 'int64_t' (aka 'long') changes value from 0.99 to 0: > -Wliteral-conversion > > Taking look at GIMPLE IR we end up w

[PATCH 4/4] Fix scaling of a sreal number.

2018-09-24 Thread marxin
The patch is addressing following LLVM warning: gcc/ipa-fnsummary.c:2745:54:Value Conversion Issue: implicit conversion from 'double' to 'int64_t' (aka 'long') changes value from 0.99 to 0: -Wliteral-conversion Taking look at GIMPLE IR we end up with: : D.107433 ={v} {CLOBBER}; sreal::s