<div dir="auto">Sounds good. It's a soft objection, mostly because if nothing else it puts us back where we were subject to some latent bugs, but perhaps not as bad as before (though I don't find having to use an assert build reassuring ;)<div dir="auto"><br></div><div dir="auto">Anyhow, go ahead and we'll figure out something else. </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 27, 2021, 7:23 PM Reid Kleckner via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk accepted this revision.<br>
rnk added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
In D100776#2703273 <<a href="https://reviews.llvm.org/D100776#2703273" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D100776#2703273</a>>, @echristo wrote:<br>
<br>
> As is mentioned there are tradeoffs around this though: a) it does make it harder to have clang generate code without a backend or llvm itself around, b) it does have a dependency when none existed.<br>
><br>
> So, if this is really causing some consternation then we can pull back and reinstate what we had, but it was a direction around solving a set of hard to find bugs.<br>
><br>
> Thoughts?<br>
<br>
I'm reading this as a soft, non-blocking objection. The concern that the layouts and prefix might get out of sync is addressed: there are asserts that they agree when the backends are linked in.<br>
<br>
So, under that interpretation, and without further guidance from @echristo, I think we should go forward. If that's not the right interpretation, we can always revert.<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D100776/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D100776/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D100776" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D100776</a><br>
<br>
</blockquote></div>