[PATCH] Add parser for the stackmap section format

Philip Reames listmail at philipreames.com
Fri Jun 26 14:49:34 PDT 2015


Even better!  Thanks.

On 06/26/2015 02:48 PM, Lang Hames wrote:
> 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 <mailto: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/c8a662e4/attachment.html>


More information about the llvm-commits mailing list