public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r13-3887] c-family: Support #pragma region/endregion [PR85487]
@ 2022-11-11 4:28 Jonathan Wakely
0 siblings, 0 replies; only message in thread
From: Jonathan Wakely @ 2022-11-11 4:28 UTC (permalink / raw)
To: gcc-cvs
https://gcc.gnu.org/g:d3fe767c16e7c528e4fc71c8a68ac14b4573d880
commit r13-3887-gd3fe767c16e7c528e4fc71c8a68ac14b4573d880
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Wed Nov 9 21:49:52 2022 +0000
c-family: Support #pragma region/endregion [PR85487]
These pragmas are used by some editors to mark regions of code for
grouping and folding. GCC should silently ignore them, rather than
giving -Wunknown-pragmas warnings.
PR c/85487
gcc/ChangeLog:
* doc/cpp/pragmas.rst (Pragmas): Document region pragmas.
gcc/c-family/ChangeLog:
* c-pragma.cc (handle_pragma_ignore): New function.
(init_pragma): Register region and endregion pragmas.
gcc/testsuite/ChangeLog:
* c-c++-common/pragma-region.c: New test.
Diff:
---
gcc/c-family/c-pragma.cc | 9 +++++++++
gcc/doc/cpp/pragmas.rst | 3 +++
gcc/testsuite/c-c++-common/pragma-region.c | 11 +++++++++++
3 files changed, 23 insertions(+)
diff --git a/gcc/c-family/c-pragma.cc b/gcc/c-family/c-pragma.cc
index b5a4b3c970f..142a46441ac 100644
--- a/gcc/c-family/c-pragma.cc
+++ b/gcc/c-family/c-pragma.cc
@@ -1403,6 +1403,12 @@ handle_pragma_message (cpp_reader *)
TREE_STRING_POINTER (message));
}
+/* Ignore a no-op pragma that GCC recognizes, but which has no effect. */
+static void
+handle_pragma_ignore (cpp_reader *)
+{
+}
+
/* Mark whether the current location is valid for a STDC pragma. */
static bool valid_location_for_stdc_pragma;
@@ -1870,6 +1876,9 @@ init_pragma (void)
c_register_pragma ("GCC", "pop_options", handle_pragma_pop_options);
c_register_pragma ("GCC", "reset_options", handle_pragma_reset_options);
+ c_register_pragma (0, "region", handle_pragma_ignore);
+ c_register_pragma (0, "endregion", handle_pragma_ignore);
+
c_register_pragma ("STDC", "FLOAT_CONST_DECIMAL64",
handle_pragma_float_const_decimal64);
diff --git a/gcc/doc/cpp/pragmas.rst b/gcc/doc/cpp/pragmas.rst
index da07cebf4a1..78277d7b623 100644
--- a/gcc/doc/cpp/pragmas.rst
+++ b/gcc/doc/cpp/pragmas.rst
@@ -119,3 +119,6 @@ GCC plugins may provide their own pragmas.
file will never be read again, no matter what. It is a less-portable
alternative to using :samp:`#ifndef` to guard the contents of header files
against multiple inclusions.
+
+``#pragma region {tokens}...``, ``#pragma endregion {tokens}...``
+ These pragmas are accepted, but have no effect.
diff --git a/gcc/testsuite/c-c++-common/pragma-region.c b/gcc/testsuite/c-c++-common/pragma-region.c
new file mode 100644
index 00000000000..8e0cbe1f16f
--- /dev/null
+++ b/gcc/testsuite/c-c++-common/pragma-region.c
@@ -0,0 +1,11 @@
+/* { dg-options "-Wunknown-pragmas" } */
+/* { dg-final { scan-assembler "code_within_region" } } */
+/* { dg-final { scan-assembler "code_within_named_region" } } */
+
+#pragma region
+void code_within_region() { }
+#pragma endregion
+
+#pragma region ignored name
+void code_within_named_region() { }
+#pragma endregion // ignored comment
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-11-11 4:28 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-11 4:28 [gcc r13-3887] c-family: Support #pragma region/endregion [PR85487] Jonathan Wakely
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).