On 09/24/14 14:38, Ilya Enkovich wrote:
2014-09-24 23:03 GMT+04:00 Jeff Law :
On 09/24/14 04:35, Steven Bosscher wrote:
According to description you quoted label marked by LABEL_PRESERVE_P
is used by some code or data. Let this use be not visible to the RTL
of a given function. It is still
2014-09-24 23:03 GMT+04:00 Jeff Law :
> On 09/24/14 04:35, Steven Bosscher wrote:
>>>
>>>
>>> According to description you quoted label marked by LABEL_PRESERVE_P
>>> is used by some code or data. Let this use be not visible to the RTL
>>> of a given function. It is still used, right? How can you
On 09/24/14 04:35, Steven Bosscher wrote:
According to description you quoted label marked by LABEL_PRESERVE_P
is used by some code or data. Let this use be not visible to the RTL
of a given function. It is still used, right? How can you remove it?
The code_label rtx is removed, but the labe
On 09/24/14 03:30, Steven Bosscher wrote:
What appears to be the case here, is that you have a label between two
basic blocks B1 and B2, and the label acts as a control flow barrier:
B1 and B2 cannot be merged. Then this should be expressed in the CFG.
Otherwise: What else prevents the merge_blo
2014-09-24 17:50 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote:
>> 2014-09-24 16:47 GMT+04:00 Steven Bosscher :
>> It is not a control flow instruction. It copies value of instruction
>> pointer into a general purpose register. Therefore REG_LABEL_OPERAND
>> se
On Wed, Sep 24, 2014 at 2:51 PM, Ilya Enkovich wrote:
> 2014-09-24 16:47 GMT+04:00 Steven Bosscher :
> It is not a control flow instruction. It copies value of instruction
> pointer into a general purpose register. Therefore REG_LABEL_OPERAND
> seems to be correct.
OK - sorry for being a bit slow
2014-09-24 16:47 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 2:30 PM, Ilya Enkovich wrote:
>> I didn't generate references separately from label. Now I found an
>> old patch and a test where this problem appeared. In this patch I
>> moved set_rip generation currently performed in ix86_e
On Wed, Sep 24, 2014 at 2:30 PM, Ilya Enkovich wrote:
> I didn't generate references separately from label. Now I found an
> old patch and a test where this problem appeared. In this patch I
> moved set_rip generation currently performed in ix86_expand_prologue
> into expand pass. And I got foll
2014-09-24 14:35 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 11:57 AM, Ilya Enkovich wrote:
>> 2014-09-24 13:30 GMT+04:00 Steven Bosscher :
Description of LABEL_PRESERVE_P says label that should always be
considered to be needed.
>>>
>>> It's more specific than that, really:
>>>
On Wed, Sep 24, 2014 at 11:57 AM, Ilya Enkovich wrote:
> 2014-09-24 13:30 GMT+04:00 Steven Bosscher :
>>> Description of LABEL_PRESERVE_P says label that should always be
>>> considered to be needed.
>>
>> It's more specific than that, really:
>>
>> @item LABEL_PRESERVE_P (@var{x})
>> In a @code{co
2014-09-24 13:30 GMT+04:00 Steven Bosscher :
> On Wed, Sep 24, 2014 at 8:41 AM, Ilya Enkovich wrote:
>> 2014-09-23 20:06 GMT+04:00 Jeff Law:
>>> On 09/23/14 10:01, Steven Bosscher wrote:
Are you sure this patch is necessary, and is not just papering over
another problem? In the past, all
On Wed, Sep 24, 2014 at 8:41 AM, Ilya Enkovich wrote:
> 2014-09-23 20:06 GMT+04:00 Jeff Law:
>> On 09/23/14 10:01, Steven Bosscher wrote:
>>> Are you sure this patch is necessary, and is not just papering over
>>> another problem? In the past, all cases I've seen where labels were
>>> removed inadv
2014-09-23 20:06 GMT+04:00 Jeff Law :
> On 09/23/14 10:01, Steven Bosscher wrote:
>>
>> On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote:
>>>
>>> On 09/19/14 13:36, Ilya Enkovich wrote:
Hi,
During my work on enabling pseudo PIC register I've found that cfg
cleaunp
>>>
On 09/23/14 10:01, Steven Bosscher wrote:
On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote:
On 09/19/14 13:36, Ilya Enkovich wrote:
Hi,
During my work on enabling pseudo PIC register I've found that cfg cleaunp
may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated
SET_RIP
On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote:
> On 09/19/14 13:36, Ilya Enkovich wrote:
>>
>> Hi,
>>
>> During my work on enabling pseudo PIC register I've found that cfg cleaunp
>> may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated
>> SET_RIP during expand pass and cfg
On 09/19/14 13:36, Ilya Enkovich wrote:
Hi,
During my work on enabling pseudo PIC register I've found that cfg cleaunp may
remove lables with LABEL_PRESERVE_P set to 1. In my case I generated SET_RIP
during expand pass and cfg cleanup removed label it used as an operand. Below
is a patch th
Hi,
During my work on enabling pseudo PIC register I've found that cfg cleaunp may
remove lables with LABEL_PRESERVE_P set to 1. In my case I generated SET_RIP
during expand pass and cfg cleanup removed label it used as an operand. Below
is a patch that fixes it. It is not actually required
17 matches
Mail list logo
