* [PATCH] Adjust CPP_FOR_BUILD
@ 2021-11-11 9:10 Pekka Seppänen
2021-12-02 21:01 ` Jeff Law
0 siblings, 1 reply; 2+ messages in thread
From: Pekka Seppänen @ 2021-11-11 9:10 UTC (permalink / raw)
To: gcc-patches
Hi.
CPP/CPPFLAGS were changed by commit
84401ce5fb4ecab55decb472b168100e7593e01f. That commit uses CPP as a
default for CPP_FOR_BUILD. Unless CPP is defined, GNU make defaults CPP
as `$(CC) -E'. Given the context, this is now incorrect, since
CC_FOR_BUILD should be used.
Fixes PR103011.
-- Pekka
gcc/Changelog:
* configure: Regenerate.
* configure.ac: For CPP_FOR_BUILD use $(CC_FOR_BUILD) -E instead of
$(CPP).
---
configure | 2 +-
configure.ac | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index 58979d6e3b1..a5eca91fb2a 100755
--- a/configure
+++ b/configure
@@ -4092,7 +4092,7 @@ if test "${build}" != "${host}" ; then
AR_FOR_BUILD=${AR_FOR_BUILD-ar}
AS_FOR_BUILD=${AS_FOR_BUILD-as}
CC_FOR_BUILD=${CC_FOR_BUILD-gcc}
- CPP_FOR_BUILD="${CPP_FOR_BUILD-\$(CPP)}"
+ CPP_FOR_BUILD="${CPP_FOR_BUILD-\$(CC_FOR_BUILD) -E}"
CXX_FOR_BUILD=${CXX_FOR_BUILD-g++}
DSYMUTIL_FOR_BUILD=${DSYMUTIL_FOR_BUILD-dsymutil}
GFORTRAN_FOR_BUILD=${GFORTRAN_FOR_BUILD-gfortran}
diff --git a/configure.ac b/configure.ac
index 550e6993b59..b8055dad573 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1334,7 +1334,7 @@ if test "${build}" != "${host}" ; then
AR_FOR_BUILD=${AR_FOR_BUILD-ar}
AS_FOR_BUILD=${AS_FOR_BUILD-as}
CC_FOR_BUILD=${CC_FOR_BUILD-gcc}
- CPP_FOR_BUILD="${CPP_FOR_BUILD-\$(CPP)}"
+ CPP_FOR_BUILD="${CPP_FOR_BUILD-\$(CC_FOR_BUILD) -E}"
CXX_FOR_BUILD=${CXX_FOR_BUILD-g++}
DSYMUTIL_FOR_BUILD=${DSYMUTIL_FOR_BUILD-dsymutil}
GFORTRAN_FOR_BUILD=${GFORTRAN_FOR_BUILD-gfortran}
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Adjust CPP_FOR_BUILD
2021-11-11 9:10 [PATCH] Adjust CPP_FOR_BUILD Pekka Seppänen
@ 2021-12-02 21:01 ` Jeff Law
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Law @ 2021-12-02 21:01 UTC (permalink / raw)
To: Pekka Seppänen, gcc-patches
On 11/11/2021 2:10 AM, Pekka Seppänen wrote:
> Hi.
>
> CPP/CPPFLAGS were changed by commit
> 84401ce5fb4ecab55decb472b168100e7593e01f. That commit uses CPP as a
> default for CPP_FOR_BUILD. Unless CPP is defined, GNU make defaults
> CPP as `$(CC) -E'. Given the context, this is now incorrect, since
> CC_FOR_BUILD should be used.
>
> Fixes PR103011.
>
> -- Pekka
>
>
> gcc/Changelog:
>
> * configure: Regenerate.
> * configure.ac: For CPP_FOR_BUILD use $(CC_FOR_BUILD) -E instead of
> $(CPP).
Thanks. Installed.
jeff
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-02 21:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-11 9:10 [PATCH] Adjust CPP_FOR_BUILD Pekka Seppänen
2021-12-02 21:01 ` Jeff Law
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).