)]}'
{
  "commit": "d45f882a1bf077f7e8e828fc9dfb14f3187142c1",
  "tree": "44f4dc1484d3014ae6ab355afc99657a9eb31bba",
  "parents": [
    "6c4a9a7659bb918c443e5aaf927c4bc178053eb1"
  ],
  "author": {
    "name": "Phil Noh",
    "email": "Phil.Noh@amd.com",
    "time": "Tue Jul 21 15:02:20 2026 -0500"
  },
  "committer": {
    "name": "mergify[bot]",
    "email": "37929162+mergify[bot]@users.noreply.github.com",
    "time": "Sat Aug 01 16:17:54 2026 +0000"
  },
  "message": "BaseTools: Fix Clang -Wtautological-overlap-compare in PcdValueInit.c\n\nFor each structured-PCD field copied via memcpy, DscBuildData.py\nemits the clamp expression, \u0027(FieldSize \u003e 0 \u0026\u0026 FieldSize \u003c ValueSize) ?\nFieldSize : ValueSize\u0027. When ValueSize \u003d\u003d 1 and FieldSize is unsigned,\nit reduces to (FieldSize \u003e 0 \u0026\u0026 FieldSize \u003c 1) - always-false comparison.\n\nClang flags it under -Wtautological-overlap-compare, and because\nPcdValueInit builds with -Werror, autogen fails and the build aborts with\n\u0027PcdValueInit.c: error: overlapping comparisons always evaluate to false\n[-Werror,-Wtautological-overlap-compare]\u0027. This is specific to Clang host.\n\nTo fix it, this update changes \u0027\u003c\u0027 to \u0027\u003c\u003d\u0027 at all five generator sites in\nDscBuildData.py (GenerateDefaultValueAssignFunction,\nGenerateInitValueFunction, GenerateCommandLineValue,\nGenerateModuleScopeValue, GenerateFdfValue). Behavior is unchanged:\nboth branches copy the same byte count when FieldSize \u003d\u003d ValueSize.\nGCC and MSVC builds are unaffected.\n\nSigned-off-by: Phil Noh \u003cPhil.Noh@amd.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "014f6d08058c36102503b2cba6c15efad1c0ddf0",
      "old_mode": 33188,
      "old_path": "BaseTools/Source/Python/Workspace/DscBuildData.py",
      "new_id": "bb9ba046ec526eba7ff2ed3adaf942aea678fa5f",
      "new_mode": 33188,
      "new_path": "BaseTools/Source/Python/Workspace/DscBuildData.py"
    }
  ]
}