[PATCH] D53975: Start adding the supporting code to perform out-of-process allocator enumeration.

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 15 05:35:58 PST 2018


delcypher added a comment.

In https://reviews.llvm.org/D53975#1298813, @kcc wrote:

> LGTM
>  You may still replace AddressSpaceView::Load with Load by implementing a one-line function inside this class. Up to you.


Thanks. I'd rather not do this right now. I'm worried that as I re-implement all my work using the new interface, that I will discover somewhere minor tweaks are required. Once all the out-of-process enumeration pieces have landed I'll be happy to a one-line function to make the syntax shorter.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D53975





More information about the llvm-commits mailing list