tejohnson accepted this revision. tejohnson added a comment. This revision is now accepted and ready to land.
lgtm In D128955#3676478 <https://reviews.llvm.org/D128955#3676478>, @aeubanks wrote: > In D128955#3676198 <https://reviews.llvm.org/D128955#3676198>, @tejohnson > wrote: > >> In D128955#3674787 <https://reviews.llvm.org/D128955#3674787>, @aeubanks >> wrote: >> >>> random question, if the old API is "legacy", are there any plans to remove >>> it? >> >> @fhahn started to work on this at some point (see >> https://bugs.llvm.org/show_bug.cgi?id=41541), but I'm not sure of the >> status. It is used by ld64 and I believe the sony toolchain too. > > if I'm seeing correctly, looks like there aren't any in-tree users of > ThinLTOCodeGenerator/LTOCodeGenerator. are those out of tree? Yes, all linker users are out of tree. Only llvm-lto is in tree for testing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128955/new/ https://reviews.llvm.org/D128955 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
