<div dir="ltr">Yeah InstSimplify really should be used anywhere ConstProp is being used, adding C bindings for it in <a href="https://reviews.llvm.org/D86764">https://reviews.llvm.org/D86764</a>.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 27, 2020 at 11:10 PM Roland Scheidegger 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">sroland added a comment.<br>
<br>
mesa/llvmpipe was using this pass, so it doesn't build anymore. If the proposed replacement is the InstSimplifyLegacyPass that's fine, however the old one was nicely available via the c interface (LLVMAddConstantPropagationPass) whereas the InstSimplifyLegacyPass is not.<br>
(I'd note that we only use very few passes, and they are perhaps not optimal, but the goal is to avoid any expensive passes while still getting some optimization. So, not really insisting on InstSimplifyLegacyPass neither if there are other suitable replacements doing roughly the same thing - it would be nice though if we'd not have to do our own c++ wrapper as we try to avoid that when possible.)<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D85159/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D85159/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D85159" rel="noreferrer" target="_blank">https://reviews.llvm.org/D85159</a><br>
<br>
</blockquote></div>