[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
Fri Nov 9 06:24:53 PST 2018


delcypher added a comment.

@kcc

> Maybe, as a bike-shedding exercise, try to find a better name for ObjectMap

Given that the `ObjectMap` abstraction doesn't necessarily deal with single objects having `Object` in the name isn't very helpful.
All the abstraction now does it map pointers from one address space into another while transparently handling the copying.

Based on those observations here are some other name ideas

- `PointerMap`
- `AddressSpaceView` (possibly shortened to `AddrSpaceView`). This name suggests renaming the `Map(...)` method to `Load(...)`.
- `VMView`. This name suggests renaming the `Map(...)` method to `Load(...)`.

Other suggestions are welcome.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D53975





More information about the llvm-commits mailing list