)]}'
{
  "commit": "7f55f264ec04959d4a6ec6b088d4bea0a0ff0dca",
  "tree": "7b4aa4e0c59d93c3204f60a5a31aa61875e2013f",
  "parents": [
    "397701f3a0a05f8569f923b283bc66b124795b4a"
  ],
  "author": {
    "name": "Michael Buch",
    "email": "michaelbuch12@gmail.com",
    "time": "Fri Nov 07 22:48:48 2025 +0000"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Fri Nov 07 22:48:48 2025 +0000"
  },
  "message": "[lldb][ClangModulesDeclVendor] Revamp error handling when loading Clang modules (#166917)\n\nInstead of propagating the errors as a `bool`+`Stream` we change the\n`ClangModulesDeclVendor` module loading APIs to use `llvm::Error`. We\nalso reword some of the diagnostics (notably removing the hardcoded\n`error:` prefix). A follow-up patch will further make the module loading\nerrors less noisy.\n\nSee the new tests for what the errors look like.\n\nrdar://164002569",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "6bab880b4d52164ce61d973b079a79a00bff85f2",
      "old_mode": 33188,
      "old_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp",
      "new_id": "75ed87baf636acfbceb2694904987996e5901006",
      "new_mode": 33188,
      "new_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp"
    },
    {
      "type": "modify",
      "old_id": "ff9ed9c27f70fa9bf423fefa844e52d1217c45c8",
      "old_mode": 33188,
      "old_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.cpp",
      "new_id": "b4e81aa21c138ddc98aa3e08577b9150e02dd94d",
      "new_mode": 33188,
      "new_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.cpp"
    },
    {
      "type": "modify",
      "old_id": "b77e2690deb069d31b05230bfa0998dd084ace40",
      "old_mode": 33188,
      "old_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp",
      "new_id": "6c5243b853ddfc6b791d2949f015234d8c586bd5",
      "new_mode": 33188,
      "new_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.cpp"
    },
    {
      "type": "modify",
      "old_id": "debf4761175b843870eae3d369809d3a3141d4f3",
      "old_mode": 33188,
      "old_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.h",
      "new_id": "043632007b7d3cd525459417aa0dbe23f406b126",
      "new_mode": 33188,
      "new_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.h"
    },
    {
      "type": "modify",
      "old_id": "e8d5ec3c7fd962fa842efd5f0d0000ffbddd9c7a",
      "old_mode": 33188,
      "old_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp",
      "new_id": "13d32b3bbc4f30e60ccad89131cd772ebae41538",
      "new_mode": 33188,
      "new_path": "lldb/source/Plugins/ExpressionParser/Clang/ClangUserExpression.cpp"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "b964e9b27e9149c630d147b3435f3ef05ee15b17",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_FromExpression.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "a5822ae4a75e732621d6cb882d1f7a9b0a609ca0",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_InvalidNestedSubmodule.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "0fda05283608e7e4f3d68540cabce3a81c624bc3",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_InvalidSearchPath.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "2552cdb4b61db38dc4c248e9efc2f0c5942dfbce",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_InvalidSubmodule.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "a50f784dd1dc2566c7a13399007fd7a6fb80f0b2",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_InvalidTopLevelModule.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "cadeb2de02c87bc04c9bb02a20aa7d4cd4b56abb",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_ModulemapParsing.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "a14f51eeba073b0b6390af710245586fa4e56c28",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_NoModule.test"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "a4fcace4f5b512ee11bc187d7affe36af0104935",
      "new_mode": 33188,
      "new_path": "lldb/test/Shell/Expr/TestClangModuleLoadError_NoModuleMap.test"
    }
  ]
}
