[Openmp-commits] [PATCH] D64218: [OpenMP][NFCI] Cleanup the target synchronization implementation

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Dec 15 11:05:33 PST 2020


JonChesterfield added a comment.

This looks like a difficult rebase. Some parts are obsolete (__kmpc_impl_active_thread_mask). Lifting runtime parameters with associated branches to compile time via the template is interesting but I wouldn't have guessed it's where we're losing most performance. Constant propagation probably does the same job with the bitcode RTL.

I'd be inclined to abandon this patch and recreate it if desired


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64218



More information about the Openmp-commits mailing list