[clang] [flang] [Clang] Remove ARCMigrate (PR #119269)

2024-12-09 Thread John McCall via cfe-commits
rjmccall wrote: Yes, I believe that's essentially just a version of the rewriter that uses a different ABI (it tries to match the non-fragile Apple ObjC runtime rather than the fragile one). https://github.com/llvm/llvm-project/pull/119269 ___ cfe-co

[clang] [flang] [Clang] Remove ARCMigrate (PR #119269)

2024-12-09 Thread via cfe-commits
Sirraide wrote: > `test/Rewrite` is for the Objective-C => MSVC-compatible C++ rewriter > (`-rewrite-objc`), which is a different feature. That feature might _also_ be > obsolete, but we haven't signed off on that yet AFAIK. I see, we were talking about this earlier today and weren’t quite sur

[clang] [flang] [Clang] Remove ARCMigrate (PR #119269)

2024-12-09 Thread John McCall via cfe-commits
rjmccall wrote: `test/Rewrite` is for the Objective-C => MSVC-compatible C++ rewriter (`-rewrite-objc`), which is a different feature. That feature might *also* be obsolete, but we haven't signed off on that yet AFAIK. https://github.com/llvm/llvm-project/pull/119269 _

[clang] [flang] [Clang] Remove ARCMigrate (PR #119269)

2024-12-09 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (Sirraide) Changes In the discussion around #116792, @rjmccall mentioned that ARCMigrate has been obsoleted and that we could go ahead and remove it from Clang, so this patch does just that. I searched for every mention of ‘ARCMT’/‘