[PATCH] D55348: Change the objc ARC optimizer to use the new objc.* intrinsics
Pete Cooper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 17 17:57:46 PST 2018
pete marked 6 inline comments as done.
pete added inline comments.
================
Comment at: lib/Transforms/ObjCARC/ARCRuntimeEntryPoints.h:92
+ return getIntrinsicEntryPoint(RetainRV,
+ Intrinsic::objc_retainAutoreleasedReturnValue);
case ARCRuntimeEntryPointKind::RetainAutorelease:
----------------
ahatanak wrote:
> Indentation looks incorrect.
Ah yeah, 80 cols so had to do it this way :(
================
Comment at: lib/Transforms/ObjCARC/ARCRuntimeEntryPoints.h:98
+ return getIntrinsicEntryPoint(RetainAutoreleaseRV,
+ Intrinsic::objc_retainAutoreleaseReturnValue);
}
----------------
ahatanak wrote:
> Indentation looks incorrect.
Also 80 cols.
================
Comment at: test/Transforms/ObjCARC/allocas.ll:12
declare i8* @objc_retainedObject(i8*)
declare i8* @objc_unretainedObject(i8*)
----------------
ahatanak wrote:
> Is there a reason we don't use @llvm.objc.retainedObject here and in test/Transforms/ObjCARC/rv.ll?
Good catch!
Looks like those are declared but unused in both tests. I'll remove them in a followup.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55348/new/
https://reviews.llvm.org/D55348
More information about the llvm-commits
mailing list