<div dir="ltr"><div>Thanks for reporting this.</div>What config this failure is using? I don't see this error in my builds. This is an intermediate step for templatelizing the code. I don't think this will be a problem for the templated headers.<div><br></div><div>-Rong</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 17, 2021 at 10:26 AM Vedant Kumar via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">vsk added a comment.<br>
<br>
Thanks for taking a look @xur. Unfortunately I needed to revert this again, as `llc` is still failing to build (see c28622fbf363d8ade9598ea6ffb88cb9a8f7604f <<a href="https://reviews.llvm.org/rGc28622fbf363d8ade9598ea6ffb88cb9a8f7604f" rel="noreferrer" target="_blank">https://reviews.llvm.org/rGc28622fbf363d8ade9598ea6ffb88cb9a8f7604f</a>>):<br>
<br>
  duplicate symbol 'llvm::SampleProfileLoaderBaseImpl::findEquivalenceClasses(llvm::Function&)' in:<br>
      tools/llc/CMakeFiles/llc.dir/llc.cpp.o<br>
      lib/libLLVMInstCombine.a(InstCombineVectorOps.cpp.o)<br>
<br>
I believe the problem is that you have definitions like SampleProfileLoaderBaseImpl::findEquivalenceClasses that get imported into multiple TU's, then conflict at link-time. These probably need to be sunk into some .cpp and built just once (would also be better for compile time).<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D96455/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96455/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D96455" rel="noreferrer" target="_blank">https://reviews.llvm.org/D96455</a><br>
<br>
</blockquote></div>