From: "Bob Feng" <bob.c.feng@intel.com>
To: devel@edk2.groups.io
Subject: [Patch 0/4 V4] Enhance Incremental Build
Date: Tue, 3 Dec 2019 16:52:43 +0800 [thread overview]
Message-ID: <20191203085247.17784-1-bob.c.feng@intel.com> (raw)
V4: Add $(DEPS_FLAGS) for rule [C-Code-File.BASE.AARCH64 ...]
V3: Change CLANG9 to CLANGPDB according to commit 14672c34bd
V2: Fixed a bug in patch 4/4.
Incremental build reduces the build time by only building
the module that need to update. Edk2 Build system is a Makefile
based build system. The incrememtal build ability is provided by
the Make program. But Edk2 build tool need to generate correct makefile
to have Make program do incremental build correctly.
The current solution in build tool to support incremental build is that build
tool find out the include file list for each source file of a module, and in module's
makefile, build tool add the include file list as the source file's dependency.
In this way Make program can decide if it need to rebuild a source code by checking
its dependency. This solution has 2 shortcommings, one is the process of finding
include list is slow, the other is this method can't handle case that
a MACRO in #include statement so the related source file is always built.
This patch provides another method to support incremental build. That is to use
c preprocessor and trim tool to generate dependency files for the source file.
This method will save much time in AutoGen phase and handle MACRO in #include correctly.
For C files:
1. MSVS.
cl.exe has a build option /showIncludes to display include files on stdout. Build tool captures
that messages and generate dependency files, .deps files.
2. CLANG and GCC
-MMD -MF build option are used to generate dependency files by preprocessor. Build tool updates the
.deps files.
For ASL files:
1. Trim find out all the included files, which are asl specific include format, and generate .trim.deps file.
2. ASL PP use c preprocessor to find out all included files with #include format and generate a .deps file
3. build tool updates the .deps file
For ASM files (.asm, .s or .nasm):
1. Trim find out all the included files, which are asm specific include format, and generate .trim.deps file.
2. ASM PP use c preprocessor to find out all included files with #include format and generate a deps file
3. build tool updates the .deps file
Build tool add "include" instruction for those deps files in the Makefile.
This patch does not support RVCT tool chain for the BZ https://bugzilla.tianocore.org/show_bug.cgi?id=1750
Feng, Bob C (4):
BaseTools: Add build option for dependency file generation
BaseTools: Generate dependent files for ASL and ASM files
BaseTools: Update build_rule.txt to generate dependent files.
BaseTools: Enhance Basetool for incremental build
BaseTools/Conf/build_rule.template | 94 ++++---
BaseTools/Conf/tools_def.template | 173 ++++++------
BaseTools/Source/Python/AutoGen/GenMake.py | 83 ++----
.../Source/Python/AutoGen/IncludesAutoGen.py | 255 ++++++++++++++++++
.../Source/Python/AutoGen/ModuleAutoGen.py | 23 ++
BaseTools/Source/Python/Trim/Trim.py | 115 ++++++--
BaseTools/Source/Python/build/build.py | 63 ++++-
7 files changed, 588 insertions(+), 218 deletions(-)
create mode 100644 BaseTools/Source/Python/AutoGen/IncludesAutoGen.py
--
2.20.1.windows.1
next reply other threads:[~2019-12-03 8:52 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-03 8:52 Bob Feng [this message]
2019-12-03 8:52 ` [Patch 1/4 V4] BaseTools: Add build option for dependency file generation Bob Feng
2019-12-03 8:52 ` [Patch 2/4 V4] BaseTools: Generate dependent files for ASL and ASM files Bob Feng
2019-12-03 8:52 ` [Patch 3/4 V4] BaseTools: Update build_rule.txt to generate dependent files Bob Feng
2019-12-03 8:52 ` [Patch 4/4 V4] BaseTools: Enhance Basetool for incremental build Bob Feng
2019-12-03 14:33 ` [edk2-devel] [Patch 0/4 V4] Enhance Incremental Build Liming Gao
2019-12-04 2:09 ` Bob Feng
-- strict thread matches above, loose matches on Subject: below --
2019-12-05 8:18 Bob Feng
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191203085247.17784-1-bob.c.feng@intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox