[llvm] r238335 - Fix PR 23499 - Avoid multiple profile counters for functions in comdat sections.

Duncan P. N. Exon Smith dexonsmith at apple.com
Wed May 27 10:04:35 PDT 2015


> On 2015 May 27, at 09:57, Diego Novillo <dnovillo at google.com> wrote:
> 
> On Wed, May 27, 2015 at 12:51 PM, Duncan P. N. Exon Smith
> <dexonsmith at apple.com> wrote:
> 
>> Testcase?
> 
> In compiler-rt. This needs linking and runtime.

Okay, but:

  - You should have mentioned that test in your commit message :).
  - You still need a test in LLVM that the right IR gets generated.
    Compiler-RT is an okay place for full-flow tests for PGO -- does
    it all work? -- but we should have a local test that this pass
    is generating the right IR.  I think the right place is
    test/Instrumentation/InstrProfiling/.

> But my git tree is busted, git svn is not working in compiler-rt, for
> me. I need to unconfuse it before I can commit.

Not exactly sure what's wrong, but IME when this doesn't fix it:

$ git pull --rebase && git svn rebase -l

... this usually does:

$ rm -rf .git/svn
$ git svn rebase -l





More information about the llvm-commits mailing list