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

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun May 3 13:50:19 PDT 2020


ABataev added a comment.

In D79186#2016737 <https://reviews.llvm.org/D79186#2016737>, @jdoerfert wrote:

> 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).


It should be a different patch.


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