<div dir="ltr">Should ConstantProp.cpp also be removed? It doesn't look like it's been worked on in over 10 years, and aside from check-llvm lit tests, the only use is in llvm/unittests/ExecutionEngine/MCJIT/MCJITAPITest.cpp via LLVMAddConstantPropagationPass.<div>(it hasn't been ported to the new pass manager which is why I'm asking)</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 27, 2020 at 6:01 PM Chris Lattner via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.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">I’m ok removing it. The only utility I can see is as a simple interprocedural pass for teaching people. I’m not sure that it is the best thing to use for that.<br>
<br>
-Chris<br>
<br>
> On Jul 27, 2020, at 3:40 AM, Florian Hahn via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> <br>
> Hi,<br>
> <br>
> As far as I know, the ipconstprop pass has not been actively used in years and ipsccp has been used instead. This has the potential for confusion and sometimes leads people to spend time finding & reporting bugs as well as updating it to work with the latest API changes.<br>
> <br>
> If there are people actively using it, I would love to hear.<br>
> <br>
> If no-ones using it, I’d propose to remove the dead code: <a href="https://reviews.llvm.org/D84447" rel="noreferrer" target="_blank">https://reviews.llvm.org/D84447</a><br>
> <br>
> Cheers,<br>
> Florian<br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>