)]}'
{
  "commit": "c49f08c1b9f28db4444aae6fbb3a6a35866a6d1f",
  "tree": "a96f4a412fbaf733e77431f0263ce67be3a4e91f",
  "parents": [
    "d440aa021928b00b319ea11a7ddd2aa868d7fd84"
  ],
  "author": {
    "name": "Geoffrey Martin-Noble",
    "email": "gcmn@google.com",
    "time": "Wed Mar 08 09:32:07 2023 -0800"
  },
  "committer": {
    "name": "GitHub",
    "email": "noreply@github.com",
    "time": "Wed Mar 08 09:32:07 2023 -0800"
  },
  "message": "Add `-ftrivial-auto-var-init\u003dpattern` to ASAN build (#12447)\n\nThis option initializes automatic variables with a recognizable pattern\r\nlikely to cause helpful crashes rather than silent undefined behavior.\r\nSee the extensive description of this option to the patch that first\r\nintroduced it to clang:\r\nhttps://reviews.llvm.org/rG14daa20be1ad89639ec209d969232d19cf698845\r\n\r\nThis isn\u0027t technically asan, but it guards against the same type of\r\nmemory errors. It\u0027s also much lower overhead, so we could consider\r\nenabling it in more builds. For release builds we might consider\r\n`-ftrivial-auto-var-init\u003dzero`, which is more likely to mask bugs but\r\nless likely to access uninitialized memory at runtime, resulting in\r\nsecurity vulnerabilities.\r\n\r\nFor now, turning it on in one build is a good step.\r\n\r\nExample failure (now fixed):\r\nhttps://github.com/openxla/iree/actions/runs/4308619652/jobs/7515099139\r\n\r\nThanks to @Maximus- for introducing us to this option and flagging the\r\nabove failure.",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "52d494b09b81ad76dfa3e294eb989692f352c0ca",
      "old_mode": 33188,
      "old_path": "build_tools/cmake/iree_setup_toolchain.cmake",
      "new_id": "6d7ab609eade2b2449d13aa5b92b31f62f7a149a",
      "new_mode": 33188,
      "new_path": "build_tools/cmake/iree_setup_toolchain.cmake"
    }
  ]
}
