<div dir="ltr">BTW, this will also remove the entire 'runtime' directory in the LLVM repository.<div><br></div><div>Runtime libraries are now being developed in compiler-rt. That is where the GCDA profiling runtime already lives and any new instrumentation based profiling runtime would also likely live there.</div>
<div><br></div><div>If the code in the existing runtime tree is needed to start building those, they can always be found in the VCS as usual.</div><div><br></div><div><br>I'm planning to submit this tomorrow morning.</div>
<div><br></div><div>-Chandler</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 12, 2013 at 8:22 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Alright, I'm ready to nuke it. Last chance to say stop.<div><br></div><div>For context of others, this has come up repeatedly: no one we know of is using EdgeProfiling.cpp, PathProfiling.cpp, and the lib/Analysis/Profile*Pass.cpp collection of tools. They haven't been updated since 2012 when Alastair Murray looked into this stuff, and both current efforts towards PGO are essentially *totally* different implementation strategies that won't benefit from this code being here.</div>

<div><br></div><div>If anything, this code is actively confusing those efforts and so I would like to garbage collect and help pave the way for new stuff. We can always recover this code from the VCS if needed in the future.</div>

<div><br></div><div>Files on the chopping block:</div><div><div>#       deleted:    include/llvm/Analysis/PathNumbering.h</div><div>#       deleted:    include/llvm/Analysis/PathProfileInfo.h</div><div>#       deleted:    include/llvm/Analysis/ProfileDataLoader.h</div>

<div>#       deleted:    include/llvm/Analysis/ProfileDataTypes.h</div><div>#       deleted:    include/llvm/Analysis/ProfileInfo.h</div><div>#       deleted:    include/llvm/Analysis/ProfileInfoLoader.h</div><div>#       deleted:    include/llvm/Analysis/ProfileInfoTypes.h</div>

<div>#       deleted:    lib/Analysis/PathNumbering.cpp</div><div>#       deleted:    lib/Analysis/PathProfileInfo.cpp</div><div>#       deleted:    lib/Analysis/PathProfileVerifier.cpp</div><div>#       deleted:    lib/Analysis/ProfileDataLoader.cpp</div>

<div>#       deleted:    lib/Analysis/ProfileDataLoaderPass.cpp</div><div>#       deleted:    lib/Analysis/ProfileEstimatorPass.cpp</div><div>#       deleted:    lib/Analysis/ProfileInfo.cpp</div><div>#       deleted:    lib/Analysis/ProfileInfoLoader.cpp</div>

<div>#       deleted:    lib/Analysis/ProfileInfoLoaderPass.cpp</div><div>#       deleted:    lib/Analysis/ProfileVerifierPass.cpp</div><div>#       deleted:    lib/Transforms/Instrumentation/EdgeProfiling.cpp</div><div>
#       deleted:    lib/Transforms/Instrumentation/OptimalEdgeProfiling.cpp</div>
<div>#       deleted:    lib/Transforms/Instrumentation/PathProfiling.cpp</div></div><div><br></div><div>Please let me know if you have any concerns. I'm likely to submit this rather soon and revert it if I hear any problems because we've already spent too much time with newcomers looking at this code, trying to do stuff with it, and then realizing it never really worked or was actively maintained.</div>

</div>
</blockquote></div><br></div>