[PATCH] D85919: [ORC] Add JITLink-compatible remote memory-manager and LLJITWithChildProcess example
Lang Hames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 13 14:10:30 PDT 2020
lhames added a comment.
In D85919#2216614 <https://reviews.llvm.org/D85919#2216614>, @sgraenitz wrote:
> I didn't see a memory manager in the new TargetProcessControl API (yet). Maybe it can also be migrated there.
>
> What's all the clang-tidy errors from pre-merge check? Haven't had any linting issues locally. Are we supposed to add entries to this clang-tidy.ignore <https://github.com/google/llvm-premerge-checks/blob/master/scripts/clang-tidy.ignore>?
I'm not sure about the pre-merge checks. Most of those looked like noise to me? Was the patch already clang-formatted?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85919/new/
https://reviews.llvm.org/D85919
More information about the llvm-commits
mailing list