This doesn't touch objc-arc-contract because that's in the codegen pipeline.
However, this does move its corresponding initialize function into initializeCodegen().
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/Transforms/ObjCARC/move-and-merge-autorelease.ll | ||
---|---|---|
1 ↗ | (On Diff #464570) | it happened to pass with just objc-arc-contract, but anyway made it closer to the original test |
Comment Actions
Test updates could have been a separate NFC so review centers on the LPM passes removal / move to codegen only.
llvm/test/Transforms/ObjCARC/contract-end-of-use-list.ll | ||
---|---|---|
1 ↗ | (On Diff #464571) | Add to single -passes= pipeline? |
llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp | ||
---|---|---|
2486 | Looks like this method is unused after this patch. 00:13:01 ../lib/Transforms/ObjCARC/ObjCARCOpts.cpp:2486:6: warning: 'void {anonymous}::ObjCARCOpt::releaseMemory()' defined but not used [-Wunused-function] 00:13:01 2486 | void ObjCARCOpt::releaseMemory() { 00:13:01 | ^~~~~~~~~~ Can we remove it or is will it be used somewhere? |
llvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp | ||
---|---|---|
2486 | sorry about that, somebody fixed this in https://reviews.llvm.org/D136665 |
Looks like this method is unused after this patch.
gcc warns with
Can we remove it or is will it be used somewhere?