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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to