[PATCH] D79186: [OPENMP]Consider 'omp_null_allocator' as a predefined allocator.

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 3 09:33:26 PDT 2020


jdoerfert added a comment.

In D79186#2013689 <https://reviews.llvm.org/D79186#2013689>, @ABataev wrote:

> In D79186#2013603 <https://reviews.llvm.org/D79186#2013603>, @jdoerfert wrote:
>
> > What would you think about moving the allocator definitions to OMPKinds.def instead of listing them explicitly? As far as I can tell we really only distinguish between predefined and user allocators, right?
>
>
> They also are used to find the base type 'omp_allocator_handle_t`. Also, what do you mean saying `instead of listing them explicitly`? Anyway, you will need to list all the predefined allocators to be able to distinguish predefined and user-defined allocators.


Sure. But we can list them once in OMPKinds.def and the use a macro wherever we need all their names (or other information).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79186/new/

https://reviews.llvm.org/D79186





More information about the cfe-commits mailing list