[PATCH] D43027: [ThinLTO] Skip BlockAddresses while replacing uses in function import
Peter Collingbourne via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 7 15:50:52 PST 2018
Not for the summary format specifically, but you can always search the web
for "yaml reference" for general information about the format. There are
also examples of valid yaml summaries in
llvm/test/Transforms/LowerTypeTests/Inputs.
Peter
On Wed, Feb 7, 2018 at 3:15 PM, <dmitry.mikulin at sony.com> wrote:
> Are there any pointers to docs to help write them?
>
> > On Feb 7, 2018, at 3:10 PM, Peter Collingbourne <peter at pcc.me.uk> wrote:
> >
> > Generally we write them by hand. Is there anything in particular that
> you are having trouble with?
> >
> > Peter
> >
> > On Wed, Feb 7, 2018 at 2:47 PM, <dmitry.mikulin at sony.com> wrote:
> > How do I generate yams summary file?
> > Tests for -lowertypetests-write-summary always read an input summary.
> > How do I generate the input?
> >
> > > On Feb 7, 2018, at 12:17 PM, Peter Collingbourne via Phabricator <
> reviews at reviews.llvm.org> wrote:
> > >
> > > pcc added a comment.
> > >
> > > In https://reviews.llvm.org/D43027#1000808, @tejohnson wrote:
> > >
> > >> I'd like @pcc to comment. I noticed there are a number of other
> places in this file where we will still be calling replaceAllUsesWith on a
> Function. How do we know which should use which method?
> > >
> > >
> > > We need to use `replaceUsesExceptBlockAddr` if the function that we
> are replacing is a function definition. As far as I can tell, in all other
> cases we are replacing function declarations.
> > >
> > >
> > >
> > > ================
> > > Comment at: llvm/test/ThinLTO/X86/blockaddr-import.ll:2
> > > +; RUN: opt -thinlto-bc %s -o %t1.bc
> > > +; RUN: llvm-lto2 run %t1.bc -r=%t1.bc,o,x -r=%t1.bc,l,plx
> -r=%t1.bc,m,plx -r=%t1.bc,d,plx -o %t2
> > > +
> > > ----------------
> > > Can this be written as an IR-level test of the pass? See for example
> `llvm/test/Transforms/LowerTypeTests/import-icall.ll`.
> > >
> > >
> > > https://reviews.llvm.org/D43027
> > >
> > >
> > >
> >
> >
> >
> >
> > --
> > --
> > Peter
>
>
--
--
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180207/1369499a/attachment.html>
More information about the llvm-commits
mailing list