public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
* [Patch] Fortran/Openmp: Use OPT_Wopenmp for gfc_match_omp_depobj warning
@ 2024-02-23 11:25 Tobias Burnus
  2024-02-23 11:50 ` Jakub Jelinek
  0 siblings, 1 reply; 2+ messages in thread
From: Tobias Burnus @ 2024-02-23 11:25 UTC (permalink / raw)
  To: gcc-patches, fortran, Jakub Jelinek

[-- Attachment #1: Type: text/plain, Size: 182 bytes --]

When checking something else, I noticed that there was one warning in 
openmp.cc that did not use OPT_Wopenmp.

I intent to commit the attached patch later today as obvious.

Tobias

[-- Attachment #2: Womp.diff --]
[-- Type: text/x-patch, Size: 919 bytes --]

Fortran/Openmp: Use OPT_Wopenmp for gfc_match_omp_depobj warning

gcc/fortran/ChangeLog:

	* openmp.cc (gfc_match_omp_depobj): Use OPT_Wopenmp
	as warning category in gfc_warning.

 gcc/fortran/openmp.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gcc/fortran/openmp.cc b/gcc/fortran/openmp.cc
index 77f6e1732f9..38de60238c0 100644
--- a/gcc/fortran/openmp.cc
+++ b/gcc/fortran/openmp.cc
@@ -4768,8 +4768,8 @@ gfc_match_omp_depobj (void)
       if (gfc_match (" ( %v ) ", &destroyobj) == MATCH_YES)
 	{
 	  if (destroyobj->symtree != depobj->symtree)
-	    gfc_warning (0, "The same depend object should be used as DEPOBJ "
-			 "argument at %L and as DESTROY argument at %L",
+	    gfc_warning (OPT_Wopenmp, "The same depend object should be used as"
+			 " DEPOBJ argument at %L and as DESTROY argument at %L",
 			 &depobj->where, &destroyobj->where);
 	  gfc_free_expr (destroyobj);
 	}

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Patch] Fortran/Openmp: Use OPT_Wopenmp for gfc_match_omp_depobj warning
  2024-02-23 11:25 [Patch] Fortran/Openmp: Use OPT_Wopenmp for gfc_match_omp_depobj warning Tobias Burnus
@ 2024-02-23 11:50 ` Jakub Jelinek
  0 siblings, 0 replies; 2+ messages in thread
From: Jakub Jelinek @ 2024-02-23 11:50 UTC (permalink / raw)
  To: Tobias Burnus; +Cc: gcc-patches, fortran

On Fri, Feb 23, 2024 at 12:25:54PM +0100, Tobias Burnus wrote:
> When checking something else, I noticed that there was one warning in
> openmp.cc that did not use OPT_Wopenmp.
> 
> I intent to commit the attached patch later today as obvious.
> 
> Tobias

> Fortran/Openmp: Use OPT_Wopenmp for gfc_match_omp_depobj warning
> 
> gcc/fortran/ChangeLog:
> 
> 	* openmp.cc (gfc_match_omp_depobj): Use OPT_Wopenmp
> 	as warning category in gfc_warning.

LGTM.

	Jakub


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-02-23 11:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-23 11:25 [Patch] Fortran/Openmp: Use OPT_Wopenmp for gfc_match_omp_depobj warning Tobias Burnus
2024-02-23 11:50 ` Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).