On Fri, 2020-12-11 at 07:51 -0500, Nathan Sidwell wrote:
>
> I'm pretty sure this is wrong.  I think the test_text in
> inclhack.def
> should be a pre-fixed string that the testsuite presumably checks is
> converted.

You're right; I've added your change from the Bugzilla and updated the
expectation.  Does the following look better?



After 92648faa1cb2 ("aix: Fixinclude") make check-fixincludes began to
fail (at least on gcc121 machine).  Fix by updating fixincludes/tests
and rerunning genfixes.

Co-developed-by: Nathan Sidwell <nat...@acm.org>

fixincludes/ChangeLog:

2020-12-11  Ilya Leoshkevich  <i...@linux.ibm.com>

        * fixincl.x: Rerun genfixes.
        * inclhack.def(aix_physadr_t): Change test_text to something
        that needs to be replaced.
        * tests/base/sys/types.h(aix_physadr_t): Add expectation.
---
 fixincludes/fixincl.x              | 4 ++--
 fixincludes/inclhack.def           | 2 +-
 fixincludes/tests/base/sys/types.h | 5 +++++
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/fixincludes/fixincl.x b/fixincludes/fixincl.x
index 21439652bce..cc17edfba0b 100644
--- a/fixincludes/fixincl.x
+++ b/fixincludes/fixincl.x
@@ -2,11 +2,11 @@
  *
  * DO NOT EDIT THIS FILE   (fixincl.x)
  *
- * It has been AutoGen-ed  October 21, 2020 at 10:43:22 AM by AutoGen 5.18.16
+ * It has been AutoGen-ed  December  9, 2020 at 11:16:08 AM by AutoGen 5.18.16
  * From the definitions    inclhack.def
  * and the template file   fixincl
  */
-/* DO NOT SVN-MERGE THIS FILE, EITHER Wed Oct 21 10:43:22 EDT 2020
+/* DO NOT SVN-MERGE THIS FILE, EITHER Wed Dec  9 11:16:08 EST 2020
  *
  * You must regenerate it.  Use the ./genfixes script.
  *
diff --git a/fixincludes/inclhack.def b/fixincludes/inclhack.def
index 80c9adfb07c..3a4cfe06542 100644
--- a/fixincludes/inclhack.def
+++ b/fixincludes/inclhack.def
@@ -731,7 +731,7 @@ fix = {
     select    = "typedef[ \t]*struct[ \t]*([{][^}]*[}][ \t]*\\*[ 
\t]*physadr_t;)";
     c_fix     = format;
     c_fix_arg = "typedef struct __physadr_s %1";
-    test_text = "typedef struct __physadr_s {";
+    test_text = "typedef       struct { random stuff } *       physadr_t;";
 };
 
 /*
diff --git a/fixincludes/tests/base/sys/types.h 
b/fixincludes/tests/base/sys/types.h
index 683b5e93ecd..7340e76b175 100644
--- a/fixincludes/tests/base/sys/types.h
+++ b/fixincludes/tests/base/sys/types.h
@@ -9,6 +9,11 @@
 
 
 
+#if defined( AIX_PHYSADR_T_CHECK )
+typedef struct __physadr_s { random stuff } *  physadr_t;
+#endif  /* AIX_PHYSADR_T_CHECK */
+
+
 #if defined( GNU_TYPES_CHECK )
 #if !defined(_GCC_PTRDIFF_T)
 #define _GCC_PTRDIFF_T
-- 
2.25.4

Reply via email to