)]}'
{
  "commit": "455e4b80f6b8d065660ced359bf836f72c656f33",
  "tree": "b7f2d63a2d292f497e48390de7c52e6085743e51",
  "parents": [
    "7c8dfbc44ea30c0695592dd80458402c7a50ed9b"
  ],
  "author": {
    "name": "Jiawei Shao",
    "email": "jiawei.shao@intel.com",
    "time": "Sat Jun 10 00:32:22 2023 +0000"
  },
  "committer": {
    "name": "Dawn LUCI CQ",
    "email": "dawn-scoped@luci-project-accounts.iam.gserviceaccount.com",
    "time": "Sat Jun 10 00:32:22 2023 +0000"
  },
  "message": "D3D12: Always skip Robustness transform on non-DBO storage buffers\n\nThis patch adds all the storage buffers and read-only storage\nbuffers without Dynamic Buffer Offset to the \u0027ignore-list\u0027 of Tint\nRobustness transform on D3D12 backend because currently in Dawn they\nwill always be bound to descriptor tables and translated into\nByteAddressBuffers, where D3D12 runtime can guarantee that\nout-of-bounds-reading will always return 0 and out-of-bound-writing\nwill always take no action.\n\nNote that currently we cannot skip robustness transform on uniform\nbuffers because in some situations FXC will report compilation error\nwhen there is OOB access to the sized array declared in cBuffer.\n\nThis CL can improve about 11% of the performance of\ndawn_perf_test.ShaderRobustnessPerf.\n\nBug: tint:1890\nChange-Id: I34623fe1fced18208f77983d3eaf89d63afb8aed\nReviewed-on: https://dawn-review.googlesource.com/c/dawn/+/136340\nReviewed-by: Ben Clayton \u003cbclayton@google.com\u003e\nCommit-Queue: Jiawei Shao \u003cjiawei.shao@intel.com\u003e\nKokoro: Kokoro \u003cnoreply+kokoro@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "9adebebe5b74f713f29c6c08e4ead23d796bead2",
      "old_mode": 33188,
      "old_path": "src/dawn/native/d3d/D3DCompilationRequest.h",
      "new_id": "dc12a32195634d47a973e61ebe567bdc3bb0e47a",
      "new_mode": 33188,
      "new_path": "src/dawn/native/d3d/D3DCompilationRequest.h"
    },
    {
      "type": "modify",
      "old_id": "ddb38b7dfa6af163647f3a1e4c1486ba1585a113",
      "old_mode": 33188,
      "old_path": "src/dawn/native/d3d/ShaderUtils.cpp",
      "new_id": "37bac70ae6af4e071a23bbaa581214108f0aa46d",
      "new_mode": 33188,
      "new_path": "src/dawn/native/d3d/ShaderUtils.cpp"
    },
    {
      "type": "modify",
      "old_id": "78d9b70dc6b599e5acaa800bd33c938b4fb774c8",
      "old_mode": 33188,
      "old_path": "src/dawn/native/d3d12/ShaderModuleD3D12.cpp",
      "new_id": "83dced96488241c8c057f1b6f6032c73d7a557ea",
      "new_mode": 33188,
      "new_path": "src/dawn/native/d3d12/ShaderModuleD3D12.cpp"
    },
    {
      "type": "modify",
      "old_id": "120d25614e8d8ad5b43a21bf31b7e98670c369dc",
      "old_mode": 33188,
      "old_path": "src/tint/ast/transform/robustness.cc",
      "new_id": "aa947485c385449941af999119c78b1448ed5253",
      "new_mode": 33188,
      "new_path": "src/tint/ast/transform/robustness.cc"
    },
    {
      "type": "modify",
      "old_id": "35804b48927a113a58a3da83febaddb49a95ea01",
      "old_mode": 33188,
      "old_path": "src/tint/ast/transform/robustness.h",
      "new_id": "7278a41084b5a4df5faa5d5becf8ed599bcb7a53",
      "new_mode": 33188,
      "new_path": "src/tint/ast/transform/robustness.h"
    },
    {
      "type": "modify",
      "old_id": "804b449cb4ff98429d8e0aded0d02eaeb822a629",
      "old_mode": 33188,
      "old_path": "src/tint/writer/hlsl/generator.h",
      "new_id": "9fbc01ccfd20deb24d0209772246509ba3532a56",
      "new_mode": 33188,
      "new_path": "src/tint/writer/hlsl/generator.h"
    },
    {
      "type": "modify",
      "old_id": "79b091c9f094d8e0470cf830ab0764616f32a3fe",
      "old_mode": 33188,
      "old_path": "src/tint/writer/hlsl/generator_impl.cc",
      "new_id": "1b30d8d56c2875b0ee65defbd249eb0c9554f787",
      "new_mode": 33188,
      "new_path": "src/tint/writer/hlsl/generator_impl.cc"
    }
  ]
}