Review request: generate DWARF pubnames under a compiler option

Eric Christopher echristo at gmail.com
Tue Feb 12 09:14:38 PST 2013


On Tue, Feb 12, 2013 at 9:13 AM, Eric Christopher <echristo at gmail.com>wrote:

>
>
> On Tue, Feb 12, 2013 at 9:06 AM, Krzysztof Parzyszek <
> kparzysz at codeaurora.org> wrote:
>
>> On 2/11/2013 4:20 PM, Eric Christopher wrote:
>>
>>>
>>> [...] Also, I'd hope that whatever way you're iterating over the list is
>>>
>>> stable. I didn't think to check that, so if it isn't please make sure it
>>> is.
>>>
>>
>> The StringMap iterator will give the same sequence for each map that has
>> been filled with the same contents and the individual insertions were done
>> in the same order.  Unless the code that inserts the names into the map
>> changes, the results should be the same.
>>
>> On a somewhat different note---is there a way to test the change to
>> llvm-dwarfdump?  I already committed it without a test, and testing it
>> would require something like a pre-existing object file to parse.  I saw
>> that there were other testcases that had object files associated with
>> them---is this the usual strategy?
>>
>>
> Yep.
>
>
To elaborate this object file should match the end-to-end testcase that you
add for the emission that you also test.

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130212/0b6c810f/attachment.html>


More information about the llvm-commits mailing list