* [PATCH, fixincludes] Fix PR 48009 53348
@ 2015-02-15 20:49 David Edelsohn
2015-02-15 21:52 ` Bruce Korb
0 siblings, 1 reply; 2+ messages in thread
From: David Edelsohn @ 2015-02-15 20:49 UTC (permalink / raw)
To: Bruce Korb; +Cc: GCC Patches
The stdlib.h header in AIX 4.3 does not correctly declare strtof with
a const char* argument. Users are building the latest releases of GCC
on AIX 4.3 The appended patch from Richard G Daniel uses fixincludes
to correct the declaration.
Okay?
Thanks, David
PR bootstrap/48009
PR bootstrap/53348
* inclhack.def (aix_strtof_const): New fix.
* fixincl.x: Regenerate.
* tests/base/inttypes.h: New test.
Index: inclhack.def
===================================================================
--- inclhack.def (revision 220717)
+++ inclhack.def (working copy)
@@ -842,6 +842,18 @@
};
/*
+ * stdlib.h on AIX 4.3 declares strtof() with a non-const first argument.
+ */
+fix = {
+ hackname = aix_strtof_const;
+ files = stdlib.h;
+ select = "((extern[ \t]+)?float[ \t]+strtof)\\(char \\*, char \\*\\*\\);
";
+ c_fix = format;
+ c_fix_arg = "%1(const char *, char **);";
+ test_text = "extern float strtof(char *, char **);";
+};
+
+/*
* sys/machine.h on AIX 4.3.3 puts whitespace between a \ and a newline
* in an otherwise harmless (and #ifed out) macro definition
*/
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH, fixincludes] Fix PR 48009 53348
2015-02-15 20:49 [PATCH, fixincludes] Fix PR 48009 53348 David Edelsohn
@ 2015-02-15 21:52 ` Bruce Korb
0 siblings, 0 replies; 2+ messages in thread
From: Bruce Korb @ 2015-02-15 21:52 UTC (permalink / raw)
To: David Edelsohn; +Cc: GCC Patches
Looks good to me.
On Sun, Feb 15, 2015 at 12:49 PM, David Edelsohn <dje.gcc@gmail.com> wrote:
> The stdlib.h header in AIX 4.3 does not correctly declare strtof with
> a const char* argument. Users are building the latest releases of GCC
> on AIX 4.3 The appended patch from Richard G Daniel uses fixincludes
> to correct the declaration.
>
> Okay?
>
> Thanks, David
>
> PR bootstrap/48009
> PR bootstrap/53348
> * inclhack.def (aix_strtof_const): New fix.
> * fixincl.x: Regenerate.
> * tests/base/inttypes.h: New test.
>
> Index: inclhack.def
> ===================================================================
> --- inclhack.def (revision 220717)
> +++ inclhack.def (working copy)
> @@ -842,6 +842,18 @@
> };
>
> /*
> + * stdlib.h on AIX 4.3 declares strtof() with a non-const first argument.
> + */
> +fix = {
> + hackname = aix_strtof_const;
> + files = stdlib.h;
> + select = "((extern[ \t]+)?float[ \t]+strtof)\\(char \\*, char \\*\\*\\);
> ";
> + c_fix = format;
> + c_fix_arg = "%1(const char *, char **);";
> + test_text = "extern float strtof(char *, char **);";
> +};
> +
> +/*
> * sys/machine.h on AIX 4.3.3 puts whitespace between a \ and a newline
> * in an otherwise harmless (and #ifed out) macro definition
> */
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-02-15 21:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-15 20:49 [PATCH, fixincludes] Fix PR 48009 53348 David Edelsohn
2015-02-15 21:52 ` Bruce Korb
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).