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

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 30 14:00:20 PDT 2020


ABataev added a comment.

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.


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