<div dir="ltr">I had given this some thought. I have a very limited knowledge of the code in this space. IIUC this only matters if the idea of linkage can change, which would happen only if a tag name needs manufacturing, which can only happen in the case we ran across (someone check my logic here?).  The other thing that factored into my thinking is that this seemed to be the only reported downside. I don't feel qualified to make a decision whether moving the code farther down would be better.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 24, 2022 at 2:04 PM David Blaikie 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">dblaikie added a comment.<br>
<br>
@aaron.ballman wouldn't mind your take on this to see if this seems sufficiently robust, tested, etc. (should I move the isExternallyVisible check even further down? So its side effects are even less impactful (maybe there are other warnings that care about this sort of thing?) )<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D121328/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D121328/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D121328" rel="noreferrer" target="_blank">https://reviews.llvm.org/D121328</a><br>
<br>
</blockquote></div>