[PATCH] gas: fix a few omissions in .cfi_label handling

[H.J. Lu]

On Thu, Feb 5, 2015 at 8:17 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>> On 05.02.15 at 16:50, <hjl.tools@gmail.com> wrote:
>> On Thu, Feb 5, 2015 at 7:45 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>>> On 05.02.15 at 16:38, <hjl.tools@gmail.com> wrote:
>>>> On Thu, Feb 5, 2015 at 7:06 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>>>> While actually starting to use that new directive, I noticed a few
>>>>> oversights of the original commit.
>>>>>
>>>>> gas/
>>>>> 2015-02-05  Jan Beulich  <jbeulich@suse.com>
>>>>>
>>>>>         * dw2gencfi.c (select_cie_for_fde): Also bail on CFI_label.
>>>>>         (cfi_change_reg_numbers): Also do nothing for CFI_label.
>>>>>         (cfi_pseudo_table): Also handle .cfi_label when not supporting
>>>>>         CFI directives.
>>>>>
>>>>
>>>> Please add some testcases.
>>>
>>> For what? The original patch added some already, and I really don't
>>> see the point of me wasting time on trying to figure out how I could
>>> demonstrate that things could go wrong without these (minor and
>>> obvious) adjustments.
>>
>> How does one know what the new directive should
>> behave?  Without the testcase, it may be broken and
>> we won't even know it.
>
> There is no new directive being added here. All that happens are a
> couple of adjustments to the code that added it recently (with - upon
> your request - a test case).
>

Doesn't your new patch add cfi_label?  If it is just a dummy,
why not just replace those dummy CFI directives with a
cfi_dummy directive?

-- 
H.J.