[PATCH] D121630: [IRLinker] make IRLinker::AddLazyFor Optional. NFC

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 11:27:36 PDT 2022


nickdesaulniers created this revision.
nickdesaulniers added reviewers: dexonsmith, tejohnson.
Herald added subscribers: ormris, steven_wu, hiraditya.
Herald added a project: All.
nickdesaulniers requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

2 of the 3 callsite of IRMover::move() pass empty lambda functions. Just
make this parameter llvm::Optional.

Came about via discussion in D120781 <https://reviews.llvm.org/D120781>. Probably worth making this change
regardless of the resolution of D120781 <https://reviews.llvm.org/D120781>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D121630

Files:
  llvm/include/llvm/Linker/IRMover.h
  llvm/lib/LTO/LTO.cpp
  llvm/lib/Linker/IRMover.cpp
  llvm/lib/Linker/LinkModules.cpp
  llvm/lib/Transforms/IPO/FunctionImport.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121630.415171.patch
Type: text/x-patch
Size: 6495 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220314/c3eb7f31/attachment.bin>


More information about the llvm-commits mailing list