[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/sandeepkosuri closed https://github.com/llvm/llvm-project/pull/82604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/alexey-bataev approved this pull request. LG https://github.com/llvm/llvm-project/pull/82604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/SunilKuravinakop updated https://github.com/llvm/llvm-project/pull/82604 >From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001 From: Sunil Kuravinakop Date: Tue, 20 Feb 2024 11:29:49 -0600 Subject: [PATCH 1/7] Accept multiple init clauses in interop directiv

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

@@ -7023,31 +7023,47 @@ void CodeGenFunction::EmitOMPInteropDirective(const OMPInteropDirective &S) { S.getSingleClause())) && "OMPNowaitClause clause is used separately in OMPInteropDirective."); - if (const auto *C = S.getSingl

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

SunilKuravinakop wrote: The recent commit also takes care of the feedback provided by Alexey. https://github.com/llvm/llvm-project/pull/82604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/SunilKuravinakop updated https://github.com/llvm/llvm-project/pull/82604 >From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001 From: Sunil Kuravinakop Date: Tue, 20 Feb 2024 11:29:49 -0600 Subject: [PATCH 1/6] Accept multiple init clauses in interop directiv

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

@@ -7023,19 +7023,25 @@ void CodeGenFunction::EmitOMPInteropDirective(const OMPInteropDirective &S) { S.getSingleClause())) && "OMPNowaitClause clause is used separately in OMPInteropDirective."); - if (const auto *C = S.getSingl

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

SunilKuravinakop wrote: Alexey, Can you please review my changes? https://github.com/llvm/llvm-project/pull/82604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/SunilKuravinakop updated https://github.com/llvm/llvm-project/pull/82604 >From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001 From: Sunil Kuravinakop Date: Tue, 20 Feb 2024 11:29:49 -0600 Subject: [PATCH 1/4] Accept multiple init clauses in interop directiv

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

@@ -7023,19 +7023,25 @@ void CodeGenFunction::EmitOMPInteropDirective(const OMPInteropDirective &S) { S.getSingleClause())) && "OMPNowaitClause clause is used separately in OMPInteropDirective."); - if (const auto *C = S.getSingl

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

@@ -7023,19 +7023,25 @@ void CodeGenFunction::EmitOMPInteropDirective(const OMPInteropDirective &S) { S.getSingleClause())) && "OMPNowaitClause clause is used separately in OMPInteropDirective."); - if (const auto *C = S.getSingl

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

@@ -7023,19 +7023,25 @@ void CodeGenFunction::EmitOMPInteropDirective(const OMPInteropDirective &S) { S.getSingleClause())) && "OMPNowaitClause clause is used separately in OMPInteropDirective."); - if (const auto *C = S.getSingl

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

@@ -7023,19 +7023,25 @@ void CodeGenFunction::EmitOMPInteropDirective(const OMPInteropDirective &S) { S.getSingleClause())) && "OMPNowaitClause clause is used separately in OMPInteropDirective."); - if (const auto *C = S.getSingl

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/SunilKuravinakop updated https://github.com/llvm/llvm-project/pull/82604 >From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001 From: Sunil Kuravinakop Date: Tue, 20 Feb 2024 11:29:49 -0600 Subject: [PATCH 1/3] Accept multiple init clauses in interop directiv

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/SunilKuravinakop updated https://github.com/llvm/llvm-project/pull/82604 >From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001 From: Sunil Kuravinakop Date: Tue, 20 Feb 2024 11:29:49 -0600 Subject: [PATCH 1/3] Accept multiple init clauses in interop directiv

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

alexey-bataev wrote: The test is required https://github.com/llvm/llvm-project/pull/82604 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

llvmbot wrote: @llvm/pr-subscribers-clang Author: None (SunilKuravinakop) Changes Modifying clang/lib/CodeGen/CGStmtOpenMP.cpp to accept multiple `init` clauses with `interop` directive. --- Full diff: https://github.com/llvm/llvm-project/pull/82604.diff 1 Files Affected: - (modified)

[clang] [OpenMP] Clang Codegen Interop : Accept multiple init (PR #82604)

https://github.com/SunilKuravinakop created https://github.com/llvm/llvm-project/pull/82604 Modifying clang/lib/CodeGen/CGStmtOpenMP.cpp to accept multiple `init` clauses with `interop` directive. >From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001 From: Sunil Kuravinakop