Tianocore/edk2.git/BaseTools, branch master

BaseTools: Add ClangToolChain build plugin

2026-09-17T17:11:50+00:00

Add a new edk2-pytools build plugin that automatically configures CLANG_BIN for CLANGPDB and CLANGDWARF tool chains. The plugin resolves the clang binary path by checking, in order: 1. CLANG_BIN environment variable 2. System PATH 3. Default LLVM install path (Windows) 4. VS-installed clang (Windows) The plugin also reports clang and linker (lld-link/lld) versions to the version aggregator for build logging. Signed-off-by: Aaron Pop

BaseTools/FirmwareStorageFormat: Fix FV ext entry struct factory functions

2026-09-13T19:05:51+00:00

The Refine_FV_EXT_ENTRY_OEM_TYPE_Header() and Refine_FV_EXT_ENTRY_GUID_TYPE_Header() functions incorrectly return an instantiation attempt of the local class with `Structure` as an argument (e.g., `return ClassName(Structure)`) instead of returning the class itself (e.g., `return ClassName`). This causes FMMT to fail with: "expected EFI_FIRMWARE_VOLUME_EXT_ENTRY instance, got _ctypes.PyCStructType" When processing FVs that contain ext header entries of type 0x01 (OEM Type) or type 0x02 (GUID Type, used for FV UI Name when FvNameString = TRUE). Fix by returning the class directly, matching the pattern used by Refine_FV_Header() which correctly returns the class for subsequent .from_buffer_copy() calls. Signed-off-by: Michael D Kinney

BaseTools: PatchCheck: Report File Line No, Not Patch

2026-09-08T17:27:21+00:00

Currently, when the GitDiffCheck runs and a failure is reported, PatchCheck reports what line of the patch failed, not the line number of the file. This is not particularly useful because a developer needs the file line number to go fix the issue. This patch updates the behavior to keep track of the file line number (based on the patch reported line numbers) and report that as where the failure occurred instead of the patch line number. Signed-off-by: Oliver Smith-Denny

BaseTools: Optimize parsing of byte-array PCD values

2026-09-06T23:50:31+00:00

This change adds a fast path for simple VOID* hexadecimal byte arrays, avoiding the generic expression tokenizer's repeated processing of large PCD values. Regression tests for both paths. It does not change the existing parser for structured and symbolic expressions. This reduces parsing time for a 28 KiB generated PCD from roughly 2 seconds to 50 milliseconds. Signed-off-by: Kun Qin

BaseTools/GenSec: Use SECTION_SIZE to read section size

2026-09-03T10:36:41+00:00

Replace the open-coded read of the 3-byte section Size[] field with the SECTION_SIZE() macro from Pi/PiFirmwareFile.h. The macro reads exactly the three Size[] bytes (endianness-safe) instead of a 4-byte load that over-reads the adjacent byte. No functional change. Verified by building BaseTools and running GenSec: python3 BaseTools/Edk2ToolsBuild.py -t GCC GenSec -s EFI_SECTION_RAW -o out.sec in.bin # in.bin: 100 bytes out.sec is 104 bytes: the 100-byte payload plus the 4-byte EFI_COMMON_SECTION_HEADER. The Size field of the header reads 0x000068, matching out.sec's size that is calculated by SECTION_SIZE. Generated with the following Coccinelle semantic patch: ```smpl @@ EFI_COMMON_SECTION_HEADER *E; typedef UINT32; @@ ( - *(UINT32 *)(E->Size) & 0x00FFFFFF + SECTION_SIZE (E) | - *(UINT32 *)E->Size & 0x00FFFFFF + SECTION_SIZE (E) ) ``` Signed-off-by: Mingjie Shen

BaseTools: Make DSC arch macro expansion owner-aware

2026-09-01T14:42:26+00:00

PR #12628 expanded architecture macros for every DSC record so component-private records match their component. A positive raw owner can also represent include provenance, so broad expansion changes records outside component scope. Expand architecture macros only for Component records and records whose owner maps to a final Component. Reset the owner mapping whenever the post-processed table is rebuilt to prevent stale ownership across DoPostProcess calls. Add regression coverage for component-private LibraryClasses and PCDs, nested and private includes, multiple architectures, unresolved macros, repeated sections, and repeated post-processing. Signed-off-by: Bob Chen (UST Global Singapore Pte Limited)

BaseTools/Python: Remove error message with email address

2026-08-28T10:04:05+00:00

Remove the "Please send email to devel@edk2.groups.io" message from the unhandled exception handlers in build.py, GenFds.py, Trim.py, and the UPT tools. Also remove the now-unused MSG_EDKII_MAIL_ADDR and MSG_SEARCH_FOR_HELP constants from Common/DataType.py and UPT/Logger/StringTable.py. Signed-off-by: Vishal Oliyil Kunnil

BaseTools/Build: Output warning message for library class mismatch

2026-08-06T04:55:34+00:00

Performs a check that will verify that the library instance implements the library specified in the dsc by ensuring a LIBRARY_CLASS definition exists in the INF [Defines] section and the value matches the library it says it is implementing. As an example, from a platform dsc file: BaseBmpSupportLib|MdeModulePkg/Library/BaseBmpSupportLib/BaseBmpSupportLib.inf BaseBmpSupportLib is supposed to be of library class BmpSupportLib, but the dsc defines it incorrectly, the warning message will be displayed during build. Signed-off-by: Aaron Pop Co-authored-by: Poncho Figueroa

BaseTools: Fix Clang -Wtautological-overlap-compare in PcdValueInit.c

2026-08-01T16:17:54+00:00

For each structured-PCD field copied via memcpy, DscBuildData.py emits the clamp expression, '(FieldSize > 0 && FieldSize < ValueSize) ? FieldSize : ValueSize'. When ValueSize == 1 and FieldSize is unsigned, it reduces to (FieldSize > 0 && FieldSize < 1) - always-false comparison. Clang flags it under -Wtautological-overlap-compare, and because PcdValueInit builds with -Werror, autogen fails and the build aborts with 'PcdValueInit.c: error: overlapping comparisons always evaluate to false [-Werror,-Wtautological-overlap-compare]'. This is specific to Clang host. To fix it, this update changes '<' to '<=' at all five generator sites in DscBuildData.py (GenerateDefaultValueAssignFunction, GenerateInitValueFunction, GenerateCommandLineValue, GenerateModuleScopeValue, GenerateFdfValue). Behavior is unchanged: both branches copy the same byte count when FieldSize == ValueSize. GCC and MSVC builds are unaffected. Signed-off-by: Phil Noh

BaseTools: iasl: Update iasl binaries to 20230628 release

2026-07-31T15:16:47+00:00

This change updates the iasl binary to the 20230628 release. The updated release also adds support for execution on ARM host machines. Signed-off-by: Kun Qin