On 2024-08-16 16:45, Jakub Jelinek wrote:
On Fri, Aug 16, 2024 at 03:51:01PM +0200, Torbjörn SVENSSON wrote:
gcc/testsuite/ChangeLog:
* g++.dg/warn/pr33738.C: Added -fno-short-enums.
* g++.dg/warn/pr33738-2.C: Duplicate g++.dg/warn/pr33738.C with
-fshort-enums and removed xfail.
Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
---
gcc/testsuite/g++.dg/warn/pr33738-2.C | 27 +++++++++++++++++++++++++++
gcc/testsuite/g++.dg/warn/pr33738.C | 3 ++-
2 files changed, 29 insertions(+), 1 deletion(-)
create mode 100644 gcc/testsuite/g++.dg/warn/pr33738-2.C
--- a/gcc/testsuite/g++.dg/warn/pr33738.C
+++ b/gcc/testsuite/g++.dg/warn/pr33738.C
@@ -1,5 +1,6 @@
// { dg-do run }
-// { dg-options "-O2 -Wtype-limits -fstrict-enums" }
+/* { dg-prune-output "use of enum values across objects may fail" } */
+// { dg-options "-O2 -Wtype-limits -fstrict-enums -fno-short-enums" }
When the test already uses // style comments, use that for the new
dg-prune-output as well.
I copy-pasted from another test case and didn't see that it used c-style
comments until after I sent the patch. I intended to fix that before
merge regardless if someone commented on it or not.
extern void link_error (void);
enum Alpha {
--
2.25.1
Ok for trunk with that nit fixed.
Jakub
Pushed as r15-2952 and r14-10594.
Kind regards,
Torbjörn
