)]}'
{
  "commit": "488ca0eacefd3bc8c7570e8ed5053f4a49451419",
  "tree": "bff9a5f033aad93665c63aada58dec379bcfd30c",
  "parents": [
    "e1bfd16dd712b0fba132cbca2f7d383e6edb0f65"
  ],
  "author": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Tue Aug 08 17:21:28 2017 -0400"
  },
  "committer": {
    "name": "David Benjamin",
    "email": "davidben@google.com",
    "time": "Mon Aug 14 19:16:25 2017 +0000"
  },
  "message": "Enable ADX in x86_64-mont*.pl.\n\nThis is a reland of https://boringssl-review.googlesource.com/18965\nwhich was reverted due to Windows toolchain problems that have since\nbeen fixed.\n\nWe have an SDE bot now and can more easily test things. We also enabled\nADX in rsaz-avx2.pl which does not work without x86_64-mont*.pl enabled.\nrsa-avx2.pl\u0027s ADX code only turns itself off so that the faster ADX code\ncan be used... but we disable it.\n\nVerified, after reverting the fix, the test vectors we imported combined\nwith Intel SDE catches CVE-2016-7055, so we do indeed have test\ncoverage. Also verified on the Windows version of Intel SDE.\n\nThanks to Alexey Ivanov for pointing out the discrepancy.\n\nSkylake numbers:\n\nBefore:\nDid 7296 RSA 2048 signing operations in 10038191us (726.8 ops/sec)\nDid 209000 RSA 2048 verify operations in 10030629us (20836.2 ops/sec)\nDid 1080 RSA 4096 signing operations in 10072221us (107.2 ops/sec)\nDid 60836 RSA 4096 verify operations in 10053929us (6051.0 ops/sec)\n\nADX consistently off:\nDid 9360 RSA 2048 signing operations in 10025823us (933.6 ops/sec)\nDid 220000 RSA 2048 verify operations in 10024339us (21946.6 ops/sec)\nDid 1048 RSA 4096 signing operations in 10006782us (104.7 ops/sec)\nDid 61936 RSA 4096 verify operations in 10088011us (6139.6 ops/sec)\n\nAfter (ADX consistently on):\nDid 10444 RSA 2048 signing operations in 10006781us (1043.7 ops/sec)\nDid 323000 RSA 2048 verify operations in 10012192us (32260.7 ops/sec)\nDid 1610 RSA 4096 signing operations in 10044930us (160.3 ops/sec)\nDid 96000 RSA 4096 verify operations in 10075606us (9528.0 ops/sec)\n\nChange-Id: I2502ce80e9cfcdea40907512682e3a6663000faa\nReviewed-on: https://boringssl-review.googlesource.com/19105\nReviewed-by: Adam Langley \u003cagl@google.com\u003e\nCommit-Queue: David Benjamin \u003cdavidben@google.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "6dfeddb2c00c2910d7d16e732797afb201d23c14",
      "old_mode": 33188,
      "old_path": "BUILDING.md",
      "new_id": "d5d3fa01ef1ba6e1d99bd76ba7678cc042521a91",
      "new_mode": 33188,
      "new_path": "BUILDING.md"
    },
    {
      "type": "modify",
      "old_id": "14a28dcd182a1915b0cbd6d2fc484b14cfd4dafd",
      "old_mode": 33261,
      "old_path": "crypto/fipsmodule/bn/asm/rsaz-avx2.pl",
      "new_id": "437c0f7f839684f773e909edb2d39d582557111b",
      "new_mode": 33261,
      "new_path": "crypto/fipsmodule/bn/asm/rsaz-avx2.pl"
    },
    {
      "type": "modify",
      "old_id": "286e0a03d37e423bf6afb5ecd7a6d1f0886cb3be",
      "old_mode": 33261,
      "old_path": "crypto/fipsmodule/bn/asm/x86_64-mont.pl",
      "new_id": "c27f65de6073bd433d678c067fecf40d98e123c6",
      "new_mode": 33261,
      "new_path": "crypto/fipsmodule/bn/asm/x86_64-mont.pl"
    },
    {
      "type": "modify",
      "old_id": "db09624418cccf43feda3bdf984765687c0c34b5",
      "old_mode": 33261,
      "old_path": "crypto/fipsmodule/bn/asm/x86_64-mont5.pl",
      "new_id": "73abb71441f01f2eb82b3fdb13703a35aaa78e31",
      "new_mode": 33261,
      "new_path": "crypto/fipsmodule/bn/asm/x86_64-mont5.pl"
    }
  ]
}