[clang] [flang] [llvm] Enable seq_cst Clause for flush Directive in OpenMP (PR #114072)

Kiran Chandramohan via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 4 06:10:07 PST 2024


https://github.com/kiranchandramohan commented:

Thanks for this patch. Could you update the title/summary of the patch to say that seq_cst is allowed in Flush since OpenMP 5.1. Also, might make sense to say why no other changes in Lowering to IR is required.

Could you add a test in flang/test/Lower/OpenMP/flush.f90 also for the Flang lowering?



https://github.com/llvm/llvm-project/pull/114072


More information about the cfe-commits mailing list