)]}'
{
  "commit": "b2eda39531cbe09b3fa8cc05cf42825519ceb2f5",
  "tree": "15fe3d5254aede7066b9abf9e5eb867bccd7edd6",
  "parents": [
    "79d9a3c5f62f65a2e537ba9ad381c665488c36a1"
  ],
  "author": {
    "name": "bjacob",
    "email": "benoitjacob@google.com",
    "time": "Mon Nov 28 14:46:57 2022 -0500"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Mon Nov 28 14:46:57 2022 -0500"
  },
  "message": "Consistently query `ExecutableTargetAttr` (#11291)\n\n#11057 added a `IREE::HAL::ExecutableTargetAttr::lookup` function which\r\nshould be the preferred way to get a `ExecutableTargetAttr`, since it\r\nhonors the overrides made possible by that PR.\r\n\r\nIt\u0027s also neatly shrinking code at call sites, and better conveys\r\nintent: before, we were querying a `ExecutableVariantOp` only to get at\r\nthe `ExecutableTargetAttr`, now we query directly the latter.\r\n\r\nThe Utils are updated to all take a `ExecutableTargetAttr`, and since\r\nthey were already implementing logic to handle optionally present\r\nfields, this further simplifies logic at call sites.\r\n\r\nSome Utils implementation functions, which were internal, are now\r\nexposed in `Utils.h`. Particularly `getConfigStringAttr`. There seemed\r\nto be no point having various place in the code reimplement that\r\nboilerplate themselves (with opportunities to mishandle optionals). A\r\nnew `getConfigIntegerAttr` is added.\r\n\r\nThis is preparation for determining tile sizes based on target info in\r\n`MaterializeEncoding`, which following #11290 we will be able to do\r\nbased on the `ExecutableTargetAttr` which we query there. It would be\r\ninconvenient, at least for testing purposes, to have to deal only with\r\n`ExecutableVariantOp` there, so the new `MaterializeEncoding` code in\r\n#11290 is using `IREE::HAL::ExecutableTargetAttr::lookup` and needs to\r\naccess helpers taking a `ExecutableTargetAttr`, which was the original\r\nmotivation for the `Utils.*` changes here.",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "0f2feb663c8cee5b15ac06eed3e7c58954f25818",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/ConvertToLLVM.cpp",
      "new_id": "35a975fff4668d57c5148029e39c230c97ba8683",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/ConvertToLLVM.cpp"
    },
    {
      "type": "modify",
      "old_id": "7a529b23e805ce9342a3edc9239cbb18739a03e6",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/KernelDispatch.cpp",
      "new_id": "d5d3522fc4e246bde15dc13af4d96701d46d6a3c",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/KernelDispatch.cpp"
    },
    {
      "type": "modify",
      "old_id": "a06562f1e090b1531c204a51ed6d4a00dea2f6b3",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/LLVMCPULowerExecutableTarget.cpp",
      "new_id": "30eff7d67a2777c924582593f79bd34313fc4fd1",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/LLVMCPULowerExecutableTarget.cpp"
    },
    {
      "type": "modify",
      "old_id": "bf26b161a6158014d6e9dde2310ef3619caa289d",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/TargetMLTransformInfo.cpp",
      "new_id": "959318f9627a8b433ab22957840f4f1e76bd5dc6",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/TargetMLTransformInfo.cpp"
    },
    {
      "type": "modify",
      "old_id": "bbdf4d3f93fae2b2f5715863112e4ff75aff9419",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/TargetMLTransformInfo.h",
      "new_id": "06ffd6f109d6327207afd669d58608491757e671",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/LLVMCPU/TargetMLTransformInfo.h"
    },
    {
      "type": "modify",
      "old_id": "03ae9f65fa53a733469fd4fe56640bc796e8c0da",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/Utils/Utils.cpp",
      "new_id": "a53f5f2b30a56bde0e4db2f439a27cb189ef210a",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/Utils/Utils.cpp"
    },
    {
      "type": "modify",
      "old_id": "506f707e921747bfd4e4904a24ded4b00cef7eea",
      "old_mode": 33188,
      "old_path": "compiler/src/iree/compiler/Codegen/Utils/Utils.h",
      "new_id": "47778b83089d394a0b533b9e39f1efd0294d486f",
      "new_mode": 33188,
      "new_path": "compiler/src/iree/compiler/Codegen/Utils/Utils.h"
    }
  ]
}
