)]}'
{
  "commit": "cbefc5e4ca7fd7aaa5d2a3aa85b30f16148c3d2f",
  "tree": "5fd37f77042352234538df3a429f0ff23c70b7c5",
  "parents": [
    "9ea11a4235b3e3521cc8bf914a27c2d0dc062db9"
  ],
  "author": {
    "name": "mtklein",
    "email": "mtklein@google.com",
    "time": "Thu Feb 11 06:00:49 2016 -0800"
  },
  "committer": {
    "name": "Commit bot",
    "email": "commit-bot@chromium.org",
    "time": "Thu Feb 11 06:00:49 2016 -0800"
  },
  "message": "Revert of SkHalfToFloat_01 / SkFloatToHalf_01 (patchset #11 id:200001 of https://codereview.chromium.org/1685133005/ )\n\nReason for revert:\nGotta fix Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD\n\nOriginal issue\u0027s description:\n\u003e SkHalfToFloat_01 / SkFloatToHalf_01\n\u003e\n\u003e These are basically inlined, 4-at-a-time versions of our existing functions,\n\u003e but cut down to avoid any work that\u0027s only necessary outside [0,1].\n\u003e\n\u003e Both f16 and f32 denorms should work fine modulo the usual ARMv7 NEON denorm\u003d\u003dzero caveat.\n\u003e\n\u003e In exchange for a little speed, f32-\u003ef16 does not round properly.\n\u003e Instead it truncates, so it\u0027s never off by more than 1 bit.\n\u003e\n\u003e Support for finite values \u003e1 or \u003c0 is straightforward to add back.\n\u003e \u003e1 might already work as-is.\n\u003e\n\u003e Getting close to _u16 performance:\n\u003e     micros   \tbench\n\u003e     261.13  \txferu64_bw_1_opaque_u16\n\u003e    1833.51  \txferu64_bw_1_alpha_u16\n\u003e    2762.32 ?\txferu64_aa_1_opaque_u16\n\u003e    3334.29  \txferu64_aa_1_alpha_u16\n\u003e     249.78  \txferu64_bw_1_opaque_f16\n\u003e    3383.18  \txferu64_bw_1_alpha_f16\n\u003e    4214.72  \txferu64_aa_1_opaque_f16\n\u003e    4701.19  \txferu64_aa_1_alpha_f16\n\u003e\n\u003e\n\u003e BUG\u003dskia:\n\u003e GOLD_TRYBOT_URL\u003d https://gold.skia.org/search2?unt\u003dtrue\u0026query\u003dsource_type%3Dgm\u0026master\u003dfalse\u0026issue\u003d1685133005\n\u003e\n\u003e Committed: https://skia.googlesource.com/skia/+/9ea11a4235b3e3521cc8bf914a27c2d0dc062db9\n\nTBR\u003djvanverth@google.com,reed@google.com,mtklein@chromium.org\n# Skipping CQ checks because original CL landed less than 1 days ago.\nNOPRESUBMIT\u003dtrue\nNOTREECHECKS\u003dtrue\nNOTRY\u003dtrue\nBUG\u003dskia:\n\nReview URL: https://codereview.chromium.org/1693443003\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "baf86fda3b394905067f0dba06db9c3da8cb7a04",
      "old_mode": 33188,
      "old_path": "src/core/SkHalf.h",
      "new_id": "7e41c6ff0c6494022c0f99649ccd88b3a46499c7",
      "new_mode": 33188,
      "new_path": "src/core/SkHalf.h"
    },
    {
      "type": "modify",
      "old_id": "6de5f1b09f871c4043aa70e001ba536c040ab98a",
      "old_mode": 33188,
      "old_path": "src/core/SkXfermodeU64.cpp",
      "new_id": "6435e6a2e2b0aa170e8a6fb5502283a9e13f8fa1",
      "new_mode": 33188,
      "new_path": "src/core/SkXfermodeU64.cpp"
    },
    {
      "type": "modify",
      "old_id": "f96d9045673a7b6313e0082e611452d3933f5576",
      "old_mode": 33188,
      "old_path": "tests/Float16Test.cpp",
      "new_id": "f437268c79faedacab7f02df01ebd2d33c917cff",
      "new_mode": 33188,
      "new_path": "tests/Float16Test.cpp"
    }
  ]
}