[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