[PATCH] MIR Serialization: Print and parse machine function names.

Alex L arphaman at gmail.com
Thu May 28 14:58:59 PDT 2015


2015-05-27 17:27 GMT-07:00 Duncan P. N. Exon Smith <dexonsmith at apple.com>:

>
> > On 2015 May 27, at 11:43, Alex Lorenz <arphaman at gmail.com> wrote:
> >
> > I rebased this patch on the reorganized initial MIR commit that landed
> in r238341.
>
> and from elsewhere in the thread:
> > Please take another look, since I rebased the patch based on reorganized
> MIR library.
>
> I don't think this needs another review, unless you're suspicious
> of your rebase?
>

I think the rebased patch is fine, I just moved things around so it's not a
clean rebase.
I will commit this later today.

Alex


>
> Let me know if you really want someone to look, otherwise LGTM
> (somewhat blindly)!
>
> >
> >
> > REPOSITORY
> >  rL LLVM
> >
> > http://reviews.llvm.org/D9841
> >
> > Files:
> >  include/llvm/CodeGen/MIRYamlMapping.h
> >  lib/CodeGen/MIRParser/MIRParser.cpp
> >  lib/CodeGen/MIRPrintingPass.cpp
> >  test/CodeGen/MIR/machine-function-missing-name.mir
> >  test/CodeGen/MIR/machine-function.mir
> >
> > EMAIL PREFERENCES
> >  http://reviews.llvm.org/settings/panel/emailpreferences/
> > <D9841.26623.patch>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150528/5c2efc8d/attachment.html>


More information about the llvm-commits mailing list