Hi Andrew, Toon,

> On 14 May 2022, at 09:09, Toon Moene <t...@moene.org> wrote:
> On 5/14/22 10:00, Iain Sandoe via Gcc-patches wrote:
>>> On 13 May 2022, at 14:58, Andrew MacLeod via Gcc-patches 
>>> <gcc-patches@gcc.gnu.org> wrote:
>>> 
>>> VRP currently searches the ssa_name list for globals to exported after it  
>>> finishes running. This change simply exports globals as they are calculated 
>>> for the final time during the DOM walk.
>>> 
>>> This avoid the occasional awkwardness of determined what ssa-names in the 
>>> list are important, as well as allowing forthcoming side-effect code to 
>>> adjust what is currently known as a global value during the walk without 
>>> affecting the values exported for the entire function.
>>> 
>>> Bootstrapped on x86_64-pc-linux-gnu with no regressions.  Pushed.
>> This (r13-436-gaf34279921f4) appears to cause or expose a problem which 
>> breaks bootstrap with in-tree MPFR on at least x86_64-linux/darwin.
>> thanks
>> Iain
>> during GIMPLE pass: threadfull
>> ../../../src/mpfr/src/sin_cos.c: In function ‘mpfr_sin_cos’:
>> ../../../src/mpfr/src/sin_cos.c:29:1: internal compiler error: in type, at 
>> value-range.h:225
>>    29 | mpfr_sin_cos (mpfr_ptr y, mpfr_ptr z, mpfr_srcptr x, mpfr_rnd_t 
>> rnd_mode)
>>       | ^~~~~~~~~~~~
>> 0x107d316 irange::type() const
>>         ../../src/gcc/value-range.h:225

> Seems to have been fixed in r13-449.

yes, indeed (thanks for the fix Andrew).

> Compare:
> 
> https://gcc.gnu.org/pipermail/gcc-testresults/2022-May/761443.html
> 
> with (r13-448):

Which, by the law of maximum cussedness, was exactly the revision I pulled for 
weekly test runs :/

sorry for the noise,
Iain


Reply via email to