[PATCH] D137083: [ObjCARC] Replace parts of ObjCARCAA with intrinsic attributes

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 5 00:39:14 PST 2022


nikic added a comment.

Thanks for the comments. I'm a bit unclear on what the right way forward here is. It sounds like the existing behavior is already incorrect (I think reporting NoModRef allows transforms that are not equivalent to a move of retain/release) and the correct action here would be to drop ARCInstKind::Retain and a few others from the getModRefInfo() implementation instead?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137083/new/

https://reviews.llvm.org/D137083



More information about the llvm-commits mailing list