testsuite: Define _POSIX_SOURCE for tests [PR113278]
Checks
Commit Message
Ok for trunk?
--
As the tests assume that fileno() is visible (only part of POSIX),
define the guard to ensure that it's visible. Currently, glibc appears
to always have this defined in C++, newlib does not.
Without this patch, fails like this can be seen:
Testing analyzer/fileno-1.c, -std=c++98
.../fileno-1.c: In function 'int test_pass_through(FILE*)':
.../fileno-1.c:5:10: error: 'fileno' was not declared in this scope
FAIL: c-c++-common/analyzer/fileno-1.c -std=c++98 (test for excess errors)
Patch has been verified on Linux.
gcc/testsuite/ChangeLog:
PR113278
* c-c++-common/analyzer/fileno-1.c: Define _POSIX_SOURCE.
* c-c++-common/analyzer/flex-with-call-summaries.c: Same.
* c-c++-common/analyzer/flex-without-call-summaries.c: Same.
Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
---
gcc/testsuite/c-c++-common/analyzer/fileno-1.c | 2 ++
gcc/testsuite/c-c++-common/analyzer/flex-with-call-summaries.c | 1 +
.../c-c++-common/analyzer/flex-without-call-summaries.c | 1 +
3 files changed, 4 insertions(+)
Comments
On Feb 15, 2024, at 9:03 AM, Torbjörn SVENSSON <torbjorn.svensson@foss.st.com> wrote:
>
> Ok for trunk?
Ok.
> gcc/testsuite/ChangeLog:
> PR113278
> * c-c++-common/analyzer/fileno-1.c: Define _POSIX_SOURCE.
> * c-c++-common/analyzer/flex-with-call-summaries.c: Same.
> * c-c++-common/analyzer/flex-without-call-summaries.c: Same.
On 2024-02-15 18:18, Mike Stump wrote:
> On Feb 15, 2024, at 9:03 AM, Torbjörn SVENSSON <torbjorn.svensson@foss.st.com> wrote:
>>
>> Ok for trunk?
>
> Ok.
Pushed as 8e8c2d2b34971bb29e74341a3efc625f1db06639.
>
>> gcc/testsuite/ChangeLog:
>> PR113278
>> * c-c++-common/analyzer/fileno-1.c: Define _POSIX_SOURCE.
>> * c-c++-common/analyzer/flex-with-call-summaries.c: Same.
>> * c-c++-common/analyzer/flex-without-call-summaries.c: Same.
@@ -1,3 +1,5 @@
+/* { dg-additional-options "-D_POSIX_SOURCE" } */
+
#include <stdio.h>
int test_pass_through (FILE *stream)
@@ -6,6 +6,7 @@
/* { dg-additional-options "-fanalyzer-call-summaries" } */
/* { dg-additional-options "-Wno-analyzer-too-complex" } */
/* { dg-additional-options "-Wno-analyzer-symbol-too-complex" } */
+/* { dg-additional-options "-D_POSIX_SOURCE" } */
/* A lexical scanner generated by flex */
@@ -4,6 +4,7 @@
/* { dg-additional-options "-fno-analyzer-call-summaries" } */
/* { dg-additional-options "-Wno-analyzer-too-complex" } */
+/* { dg-additional-options "-D_POSIX_SOURCE" } */
/* A lexical scanner generated by flex */