Implementation of OpenMP task constructs

[eZWALT]

Hey everyone, just wanted to give you a heads up that I'll be diving into implementing various task constructs of OpenMP over the next three months. This includes things like taskwait, task, taskloop, taskgroup, and more. @fabianmcg and I are already coordinating efforts, but if anyone else wants to jump in and work on a different construct, feel free to use this thread to chat about it.

Thanks you for this great community, I'm kicking things off by tackling taskwait this weekend.

[bcardosolopes]

This is great! Thanks @eZWALT and @fabianmcg for driving this, looking forward to patch review!

[bcardosolopes]

I cannot find your comment anymore, but this boils down to work that needs to be done, `CIRGenerator.cpp:164!`: ``` // For OpenMP emit declare reduction functions, if required. if (astCtx->getLangOpts().OpenMP) { llvm_unreachable("NYI"); } ```

…

On Sun, Mar 10, 2024 at 10:48 AM Walter J.T.V ***@***.***> wrote: I am seeking insights regarding an issue encountered during testing. When including the <omp.h> library in the test code as shown below: #include <omp.h> int main(){ #pragma omp parallel{ } } I observed unusual exceptions. This inclusion leads to the CIRGenerator.cpp file reaching a conditional statement that terminates with a llvm_unreachable. While I understand this may be expected behavior, I lack sufficient context to fully comprehend the issue. Could someone explain why the code functions without the inclusion of <omp.h> but raises an exception when included? This is the most relevant part of the error, if it helps: NYI UNREACHABLE executed at /home/walterjtv/Escritorio/Q8/TFG/clangir/clang/lib/CIR/CodeGen/CIRGenerator.cpp:164! PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace, preprocessed source, and associated run script. Stack dump: 0. Program arguments: /home/walterjtv/Escritorio/Q8/TFG/clangir-install/bin/clang-19 -fopenmp -fclangir-enable -emit-cir 2_basic_parallel.c -o 2_basic_parallel.cir 1. /usr/include/x86_64-linux-gnu/bits/types.h:155:26: current parser token '__fsid_t' 2. /usr/include/x86_64-linux-gnu/bits/types.h:155:12 <Spelling=/usr/include/x86_64-linux-gnu/bits/typesizes.h:73:24>: parsing struct/union body 'struct (unnamed at /usr/include/x86_64-linux-gnu/bits/types.h:155:12)' 3. /usr/include/x86_64-linux-gnu/bits/types.h:155:12 <Spelling=/usr/include/x86_64-linux-gnu/bits/typesizes.h:73:24>: CIR generation of declaration 'struct (unnamed at /usr/include/x86_64-linux-gnu/bits/types.h:155:12)' Thank you for your support. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: ***@***.***>

-- Bruno Cardoso Lopes http://www.brunocardoso.cc

[eZWALT]

Excuse me, I removed the commented because I thought that opening an issue would be more helpful. I'm pushing a PR in some minutes including implementations of taskwait, taskyield and barrier (Since this 3 have a symmetrical implementation).