--- Comment #4 from pinskia at gcc dot gnu dot org 2006-05-05 16:16 ---
Then the real question is why do you think this is a bug?
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from gary at intrepid dot com 2006-05-05 16:37 ---
(In reply to comment #4)
Then the real question is why do you think this is a bug?
1. it is a bug to create temporaries and assert 'volatile' on them
2. there is code in create_tmp_var_raw() that creates a type with
--- Comment #5 from gary at intrepid dot com 2006-05-05 16:37 ---
(In reply to comment #4)
Then the real question is why do you think this is a bug?
1. it is a bug to create temporaries and assert 'volatile' on them
Why do you say that?
2. there is code in
--- Comment #6 from pinskia at physics dot uc dot edu 2006-05-05 16:39
---
Subject: Re: create_tmp_var_raw (gimplify.c) inadventently asserts 'volatile'
on temps
--- Comment #5 from gary at intrepid dot com 2006-05-05 16:37 ---
(In reply to comment #4)
Then the
--- Comment #7 from gary at intrepid dot com 2006-05-05 16:58 ---
Consider the following:
volatile int j;
void p()
{
++j;
}
Gimplify in its present form might transform the statement above into
something like the following:
volatile int j;
void p()
{
volatile int T1;
T1 = j +
--- Comment #8 from pinskia at gcc dot gnu dot org 2006-05-05 17:10 ---
(In reply to comment #7)
Consider the following:
volatile int j;
void p()
{
++j;
}
Gimplify in its present form might transform the statement above into
something like the following:
It will never get
--- Comment #9 from gary at intrepid dot com 2006-05-05 17:55 ---
Your issue with ++j not using incr is not related to the gimplifier at all as
it also happens in every compiler version since at least 2.95.3.
I agree that is unrelated. I didn't file this bug thinking that it would
I haven't looked into the rev. history, to see why/when this fix was made,
but will ask the hypothetical: was this fix made to workaround the
misbehavior in create_tmp_var_raw()? Note that create_tmp_var_raw()
is exported from gimplify.c and appears to be called from quite a few
places.
--- Comment #10 from dberlin at gcc dot gnu dot org 2006-05-05 18:06
---
Subject: Re: create_tmp_var_raw (gimplify.c)
inadventently asserts 'volatile' on temps
I haven't looked into the rev. history, to see why/when this fix was made,
but will ask the hypothetical: was
--- Comment #11 from rth at gcc dot gnu dot org 2006-05-06 01:17 ---
(In reply to comment #9)
I haven't looked into the rev. history, to see why/when this fix was made,
but will ask the hypothetical: was this fix made to workaround the
misbehavior in create_tmp_var_raw()?
One could
--- Comment #12 from gary at intrepid dot com 2006-05-06 01:42 ---
Given the above, I suggest that my bug report be marked closed,
because the problem I was seeing in 4.0.1 is fixed by the
change to create_tmp_from_val that passes in TYPE_MAIN_VARIANT().
Note that the use of the main
11 matches
Mail list logo