[llvm-dev] Removing IPConstantPropagation.cpp
Arthur Eubanks via llvm-dev
llvm-dev at lists.llvm.org
Mon Jul 27 18:11:51 PDT 2020
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
(it hasn't been ported to the new pass manager which is why I'm asking)
On Mon, Jul 27, 2020 at 6:01 PM Chris Lattner via llvm-dev <
llvm-dev at lists.llvm.org> wrote:
> 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.
> > On Jul 27, 2020, at 3:40 AM, Florian Hahn via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
> > Hi,
> > 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.
> > If there are people actively using it, I would love to hear.
> > If no-ones using it, I’d propose to remove the dead code:
> > Cheers,
> > Florian
> > _______________________________________________
> > LLVM Developers mailing list
> > llvm-dev at lists.llvm.org
> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev