[llvm-dev] Removing IPConstantPropagation.cpp

Florian Hahn via llvm-dev llvm-dev at lists.llvm.org
Thu Jul 30 04:41:05 PDT 2020


Thanks for all the comments. I push a commit removing the pass: e77624a . It is easy enough to bring it back if we discover some crucial use cases for it.

Cheers,
Florian

> On Jul 28, 2020, at 16:15, Florian Hahn via llvm-dev <llvm-dev at lists.llvm.org> wrote:
> 
> 
> 
>> On Jul 28, 2020, at 02:11, Arthur Eubanks <aeubanks at google.com> wrote:
>> 
>> 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.
>> (it hasn't been ported to the new pass manager which is why I'm asking)
> 
> 
> If it’s unused/not maintained I think we should also remove it or move it to the examples directory with some additional material, if it is intended to illustrate a simple pass.
> 
> Cheers,
> Florian
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200730/13387c7c/attachment.html>


More information about the llvm-dev mailing list