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 | 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 | 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
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?