[llvm-dev] Removing IPConstantPropagation.cpp

[Johannes Doerfert via llvm-dev]

Functionality is basically subsumed by IPSCCP (and the Attributor).

It is also not run by O3.


On 7/27/20 11:15 AM, Ehud Katz wrote:
> I don't know this pass very well, but may I ask, why isn't it used? I mean,
> is this transformation already implemented as part of another (used)?
>
> On Mon, 27 Jul 2020 at 16:30 Johannes Doerfert via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>
>> On 7/27/20 5:40 AM, Florian Hahn via llvm-dev 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:
>> https://reviews.llvm.org/D84447
>>
>> +1 for removal
>>
>>
>>
>>> 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
>> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>>