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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
