On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law <l...@redhat.com> 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 cleanup removed label it used as an
>> operand.  Below is a patch that fixes it.  It is not actually required for
>> our latest PIC related patch but still seems to make sense.
>>
>> Bootstrapped and tested on linux-x86_64.
>>
>> Thanks,
>> Ilya
>> --
>> 2014-09-19  Ilya Enkovich  <ilya.enkov...@intel.com>
>>
>>         * cfgcleanup.c (try_optimize_cfg): Do not remove label
>>         with LABEL_PRESERVE_P flag set.
>
> OK.  Please install.
>
> Note for those not following the x86 32 bit PIC register discussion, I asked
> Ilya to submit this separately.  It was something an earlier version of his
> patch triggered and it stood out as something that ought to be fixed
> regardless of the final form of the PIC register changes that are in
> progress.

Jeff,

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 inadvertently were caused by incorrect reference counting or
missing REG_LABEL_* notes.

Did the label use count drop to zero? Is there a REG_LABEL_TARGET note
for the label operand?

Ciao!
Steven

Reply via email to