https://github.com/ravurvi20 created
https://github.com/llvm/llvm-project/pull/104777
This Pull Request introduces the `__nameof` operator, enabling users to obtain
the symbolic name of any enumerator value. When the `__nameof` operator is
used, the compiler will return the fully qualified nam
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/2] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/3] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/3] default clause replaced by otherwise clause for
metadirective
ravurvi20 wrote:
> Thank you for this patch. I wonder: was this spelling change introduced in
> 5.2? If so, does the compiler after this patch still support the 5.1 spelling
> when providing this as the OpenMP version (`-fopenmp-version=5.1`)
Yes, otherwise clause was introduced in OpenMP 5.2
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/4] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/5] default clause replaced by otherwise clause for
metadirective
ravurvi20 wrote:
@mjklemm I addressed the changes that you requested. Kindly review it and let
me know if any further modifications are required.
https://github.com/llvm/llvm-project/pull/125648
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/ravurvi20 edited
https://github.com/llvm/llvm-project/pull/125035
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ravurvi20 created
https://github.com/llvm/llvm-project/pull/124550
This PR replaces the `default` clause with the `otherwise` clause for the
`metadirective` in OpenMP. The `otherwise` clause serves as a fallback
condition when no directive from the `when` clauses is selected
https://github.com/ravurvi20 closed
https://github.com/llvm/llvm-project/pull/124550
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ravurvi20 converted_to_draft
https://github.com/llvm/llvm-project/pull/124550
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ravurvi20 created
https://github.com/llvm/llvm-project/pull/125035
This PR replaces the `default` clause with the `otherwise` clause for the
`metadirective` in OpenMP. The `otherwise` clause serves as a fallback
condition when no directive from the `when` clauses is selected
https://github.com/ravurvi20 reopened
https://github.com/llvm/llvm-project/pull/124550
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ravurvi20 closed
https://github.com/llvm/llvm-project/pull/124550
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ravurvi20 wrote:
ping @jplehr
https://github.com/llvm/llvm-project/pull/125648
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ravurvi20 wrote:
ping @mjklemm
https://github.com/llvm/llvm-project/pull/125648
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/6] default clause replaced by otherwise clause for
metadirective
ravurvi20 wrote:
> Did anybody run the OpenMP VV suite before and after this change? Curious to
> know if that improves results.
The changes I implemented are related to parsing, specifically addressing the
deprecation of the default clause. These modifications do not impact the
results, as t
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/125648
>From 189dd3cc2230ea5752969f02f119b6ee30e3df69 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 4 Feb 2025 01:35:41 -0600
Subject: [PATCH 1/6] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 created
https://github.com/llvm/llvm-project/pull/128640
This PR replaces the `default` clause with the `otherwise` clause for the
`metadirective` in OpenMP. The `otherwise` clause serves as a fallback
condition when no directive from the when clauses is selected.
ravurvi20 wrote:
Previous commit was reverted due to a failing test case, which has been
resolved now. Parsing changes have been approved in the previous PR.
[125648](https://github.com/llvm/llvm-project/pull/125648)
https://github.com/llvm/llvm-project/pull/128640
_
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH] default clause replaced by otherwise clause for metadirective
---
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH] default clause replaced by otherwise clause for metadirective
---
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH] default clause replaced by otherwise clause for metadirective
---
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/3] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/2] default clause replaced by otherwise clause for
metadirective
ravurvi20 wrote:
@mjklemm could you please approve this PR. There was minor change which has
been fixed now.
https://github.com/llvm/llvm-project/pull/128640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
@@ -2759,6 +2759,19 @@ StmtResult
Parser::ParseOpenMPDeclarativeOrExecutableDirective(
OpenMPClauseKind CKind = Tok.isAnnotation()
? OMPC_unknown
: getOpenMPClauseKind(PP.getSpelling(Tok));
+ // C
@@ -2759,6 +2759,19 @@ StmtResult
Parser::ParseOpenMPDeclarativeOrExecutableDirective(
OpenMPClauseKind CKind = Tok.isAnnotation()
? OMPC_unknown
: getOpenMPClauseKind(PP.getSpelling(Tok));
+ // C
@@ -2759,6 +2759,19 @@ StmtResult
Parser::ParseOpenMPDeclarativeOrExecutableDirective(
OpenMPClauseKind CKind = Tok.isAnnotation()
? OMPC_unknown
: getOpenMPClauseKind(PP.getSpelling(Tok));
+ // C
@@ -2759,6 +2759,18 @@ StmtResult
Parser::ParseOpenMPDeclarativeOrExecutableDirective(
OpenMPClauseKind CKind = Tok.isAnnotation()
? OMPC_unknown
: getOpenMPClauseKind(PP.getSpelling(Tok));
+ // C
@@ -2759,6 +2759,18 @@ StmtResult
Parser::ParseOpenMPDeclarativeOrExecutableDirective(
OpenMPClauseKind CKind = Tok.isAnnotation()
? OMPC_unknown
: getOpenMPClauseKind(PP.getSpelling(Tok));
+ // C
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/5] default clause replaced by otherwise clause for
metadirective
ravurvi20 wrote:
Thankyou. Updated OpenMPSupport.rst file.
https://github.com/llvm/llvm-project/pull/128640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/7] default clause replaced by otherwise clause for
metadirective
@@ -1,105 +1,33 @@
-// RUN: %clang_cc1 -verify -fopenmp -x c++ -triple x86_64-unknown-linux
-emit-llvm %s -o - | FileCheck %s
-// expected-no-diagnostics
-
+// RUN: %clang_cc1 -verify -fopenmp -fopenmp-version=52 -DOMP52 -triple
x86_64-unknown-unknown -emit-llvm %s -o - | FileCh
@@ -1660,6 +1660,10 @@ def err_omp_expected_colon : Error<"missing ':' in %0">;
def err_omp_missing_comma : Error< "missing ',' after %0">;
def err_omp_expected_context_selector
: Error<"expected valid context selector in %0">;
+def err_omp_unknown_clause
+: Error<"expe
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/6] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/6] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/7] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/9] default clause replaced by otherwise clause for
metadirective
ravurvi20 wrote:
@alexey-bataev I have pushed the requested changes. Could you please approve
this PR.
https://github.com/llvm/llvm-project/pull/128640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
@@ -1660,6 +1660,10 @@ def err_omp_expected_colon : Error<"missing ':' in %0">;
def err_omp_missing_comma : Error< "missing ',' after %0">;
def err_omp_expected_context_selector
: Error<"expected valid context selector in %0">;
+def err_omp_unknown_clause
r
@@ -1660,6 +1660,10 @@ def err_omp_expected_colon : Error<"missing ':' in %0">;
def err_omp_missing_comma : Error< "missing ',' after %0">;
def err_omp_expected_context_selector
: Error<"expected valid context selector in %0">;
+def err_omp_unknown_clause
+: Error<"expe
@@ -78,10 +78,7 @@ void foo(void) {
for (int i = 0; i < 16; i++)
;
-#pragma omp metadirective when(user = {condition(0)} \
- : parallel for) otherwise()
- for (int i=0; i<10; i++)
-;
+
ravurvi20 wrote:
Since for versi
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/4] default clause replaced by otherwise clause for
metadirective
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/4] default clause replaced by otherwise clause for
metadirective
@@ -1660,6 +1660,10 @@ def err_omp_expected_colon : Error<"missing ':' in %0">;
def err_omp_missing_comma : Error< "missing ',' after %0">;
def err_omp_expected_context_selector
: Error<"expected valid context selector in %0">;
+def err_omp_unknown_clause
+: Error<"expe
https://github.com/ravurvi20 updated
https://github.com/llvm/llvm-project/pull/128640
>From 6ebd5991788608fbd104ea9c23230912044462d3 Mon Sep 17 00:00:00 2001
From: Urvi Rav
Date: Tue, 25 Feb 2025 00:49:07 -0600
Subject: [PATCH 1/5] default clause replaced by otherwise clause for
metadirective
50 matches
Mail list logo
