)]}'
{
  "commit": "c2e8d016f5ae58b2df72db99250b31882b42b8ff",
  "tree": "d5311106ffd0cc017d181d0d9f1fc3d7d38d9bf5",
  "parents": [
    "a306b1b908cacf49d17cf77437d38da98ca5edcf"
  ],
  "author": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Sun Dec 30 02:58:55 2018 +0000"
  },
  "committer": {
    "name": "CQ bot account: commit-bot@chromium.org",
    "email": "commit-bot@chromium.org",
    "time": "Wed Jan 02 23:47:34 2019 +0000"
  },
  "message": "Fix beeu_mod_inverse_vartime CFI annotations and preamble.\n\nThis was also caught by the in-progress unwind tester. There are two\nissues here.  First, .cfi_endproc must come after ret to fully cover the\nfunction.  More importantly, this function is confused about whether it\nhas a frame pointer or not.\n\nIt looks like it does (movq %rsp, %rbp), and annotates accordingly, but\nit does not actually use the frame pointer. It cannot. $y4 is rbp and\ngets clobbered immediately after the preamble!\n\nRemove this instruction and align the CFI annotations with a\nframe-pointer-less function.\n\nBug: 181\nChange-Id: I47b5f9798b3bcee1748e537b21c173d312a14b42\nReviewed-on: https://boringssl-review.googlesource.com/c/33947\nReviewed-by: Adam Langley \u003cagl@google.com\u003e\nCommit-Queue: David Benjamin \u003cdavidben@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "12b9f5af2e9db7f6e799f084495cb2828c8cab89",
      "old_mode": 33188,
      "old_path": "crypto/fipsmodule/ec/asm/p256_beeu-x86_64-asm.pl",
      "new_id": "0bb65477ac60094e7b604d5fb36f08058d89a308",
      "new_mode": 33188,
      "new_path": "crypto/fipsmodule/ec/asm/p256_beeu-x86_64-asm.pl"
    }
  ]
}