[PATCH] Add parser for the stackmap section format

Lang Hames lhames at gmail.com
Fri Jun 26 14:48:03 PDT 2015


Hi Philip,

I'm tidying my flyweight version up and writing test cases right now. I
expect to have it in tree this afternoon. Apologies for the delay.

Cheers,
Lang.


On Fri, Jun 26, 2015 at 1:47 PM, Philip Reames <listmail at philipreames.com>
wrote:

> Unless anyone objects, I'm going to check in a cleaned up version of this
> change with style comments addressed and a use in objdump.  I agree that
> the flyweight parser is probably a better long term approach, but this a)
> works, and b) is incremental progress towards that goal.  We can adapt the
> parser once it's in and tested.
>
> If anyone wants to take lead on the flyweight style parser, I'd welcome
> that.  It's probably not something I'll get to short term. I am very happy
> to review patches.
>
> Philip
>
>
> On 06/24/2015 03:27 PM, Pat Gavlin wrote:
>
>> I prefer Lang's approach as well--I certainly wouldn't mind having this
>> checked in as an intermediate step, however.
>>
>>
>> http://reviews.llvm.org/D10377
>>
>> EMAIL PREFERENCES
>>    http://reviews.llvm.org/settings/panel/emailpreferences/
>>
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150626/c00d68a8/attachment.html>


More information about the llvm-commits mailing list