[clang] [lld] [llvm] Run ObjCContractPass in Distributed Thin-LTO Pipeline (PR #92331)

Steven Wu via llvm-commits llvm-commits at lists.llvm.org
Mon May 20 15:10:30 PDT 2024


cachemeifyoucan wrote:

> I couldn't find this check you're alluding to, so I added my own. If the check does exist, could you point me to it? If not, does mine look reasonable? Thanks.

I was answering from my memory, now I have to check. We have this function `ModuleHasARC` which is used in other ARC optimization passes. It used to be in arc-contract pass too but it seems to be dropped by https://reviews.llvm.org/D92808

It is a good idea to added it back. Maybe by reusing the same function after double check the context of the old patch. @ahatanaka might have idea if there is a reason to drop the guard.

https://github.com/llvm/llvm-project/pull/92331


More information about the llvm-commits mailing list