)]}'
{
  "commit": "a8f36ccb2a24ef4abcbff58b2f36ce2deacefd54",
  "tree": "1e75c5521115fe9edc34b40cb191bec9a17d8ae1",
  "parents": [
    "1bc571918422612d2ce4e3621955d94eca37aa24"
  ],
  "author": {
    "name": "Zachary Yedidia",
    "email": "zyedidia@gmail.com",
    "time": "Tue Apr 14 10:59:19 2026 -0700"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Tue Apr 14 10:59:19 2026 -0700"
  },
  "message": "[AArch64] Check Subtarget via STI in getInstSizeInBytes (#192089)\n\nThe InstSizes test (`llvm/unittests/Target/AArch64/InstSizes.cpp`)\ndestroys the Subtarget field early (`ST` created on the stack in\n[`createInstrInfo`](https://github.com/llvm/llvm-project/blob/40a585e742ed6b28306d7511380079325ba1a003/llvm/unittests/Target/AArch64/InstSizes.cpp#L32)),\ncausing a use-after-free if it is used in `getInstSizeInBytes`. This\ncauses a failure when running the test with hwasan (reported by build\nbot). To fix this, this PR switches to using `STI` instead of\n`Subtarget` in `getInstSizeInBytes` for checking `isLFI`, which survives\nfor the lifetime of the test.\n\nI think fixing the test itself (the root of the issue, as far as I can\ntell) would be more involved. Perhaps I should open an issue for it\nthough?\n\nI have tested the fix on an AArch64 machine with hwasan to confirm that\nit resolves the issue.",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "3ca851ef83d2761239489f60c769d803069faabd",
      "old_mode": 33188,
      "old_path": "llvm/lib/Target/AArch64/AArch64InstrInfo.cpp",
      "new_id": "9d805dad07c1cadc4954ec639a75f6d89ae49ffc",
      "new_mode": 33188,
      "new_path": "llvm/lib/Target/AArch64/AArch64InstrInfo.cpp"
    }
  ]
}