Opened 16 years ago

Closed 16 years ago

#2363 closed task (fixed)

Inconsistencies concerning "bootstrapping" in GCC instructions

Reported by: chris@… Owned by: bdubbs@…
Priority: normal Milestone: 6.5
Component: Book Version: SVN
Severity: normal Keywords:
Cc:

Description

In the Chapter 6 GCC instructions, it says...

For x86 machines, the bootstrap build performed in Section 5.10, “GCC-4.3.3 - Pass 2” built GCC with the -fomit-frame-pointer compiler flag. Non-bootstrap builds omit this flag by default, so apply the following sed to use it in order to ensure consistent compiler builds:

This is incorrect, as GCC Pass 2 is not built bootstrapped, so the first sentence of that paragraph should be altered to reflect that. Further, as it is, GCC is never bootstrapped at any point in the book, so it is never explained what "bootstrapping" means for GCC. I believe that the explanation of what "bootstrapping" means for GCC should be added to the explanation for the --disable-bootstrap switch in GCC Pass 2. Also, the text about the --disable-bootstrap switch, "Bootstrapping the compiler is now the default for GCC." should be amended to specify "for native builds." I've attached a patch with these suggested changes, as well as a little text shuffling between Chapters 5 and 6, with the description of "bootstrapping" copied from a previous LFS version.

Also, I didn't include this in the patch, but would I be right in assuming that the case statement that adds -fomit-frame-pointer only for x86 in Chapter 6 should also be used in Pass 2?

Attachments (1)

gcc-update.patch (3.2 KB ) - added by chris@… 16 years ago.
Update GCC instructions

Download all attachments as: .zip

Change History (5)

by chris@…, 16 years ago

Attachment: gcc-update.patch added

Update GCC instructions

comment:1 by Matthew Burgess, 16 years ago

Milestone: 7.0

comment:2 by bdubbs@…, 16 years ago

Milestone: 7.06.5

comment:3 by bdubbs@…, 16 years ago

Owner: changed from lfs-book@… to bdubbs@…
Status: newassigned

comment:4 by bdubbs@…, 16 years ago

Resolution: fixed
Status: assignedclosed

Fixed at revision 8883.

Note: See TracTickets for help on using tickets.