|
|
Chromium Code Reviews
DescriptionSupport saturation overrides in saturated_cast
This is a requirement for adding a full saturated type implementation.
BUG=672489
TBR=eae
NOTRY=true
Committed: https://crrev.com/71b669adca7b10a7fcecec151ec8b1c66a4c90a0
Cr-Commit-Position: refs/heads/master@{#439274}
Patch Set 1 #
Messages
Total messages: 22 (11 generated)
The CQ bit was checked by jschuh@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
I had a merge error that broke the android builders. So, I'm relanding this as a new CL with the merge error fixed. The original CL and history is here (with this change posted to make the diff easy): https://codereview.chromium.org/2578613002/
RS-LGTM
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...) android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...) android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
The CQ bit was checked by jschuh@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by jschuh@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
Description was changed from ========== Support saturation overrides in saturated_cast This is a requirement for adding a full saturated type implementation. BUG=672489 TBR=eae ========== to ========== Support saturation overrides in saturated_cast This is a requirement for adding a full saturated type implementation. BUG=672489 TBR=eae NOTRY=true ==========
The CQ bit was checked by jschuh@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch.
Bot data: {"patchset_id": 1, "attempt_start_ts": 1481936574857960, "parent_rev":
"5583da85a0558df44981cd978f0841b1e6c56743", "commit_rev":
"bc23fc6c0cc1e46321afc55e8579279a52041d39"}
Message was sent while issue was closed.
Description was changed from ========== Support saturation overrides in saturated_cast This is a requirement for adding a full saturated type implementation. BUG=672489 TBR=eae NOTRY=true ========== to ========== Support saturation overrides in saturated_cast This is a requirement for adding a full saturated type implementation. BUG=672489 TBR=eae NOTRY=true Review-Url: https://codereview.chromium.org/2579363002 ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Support saturation overrides in saturated_cast This is a requirement for adding a full saturated type implementation. BUG=672489 TBR=eae NOTRY=true Review-Url: https://codereview.chromium.org/2579363002 ========== to ========== Support saturation overrides in saturated_cast This is a requirement for adding a full saturated type implementation. BUG=672489 TBR=eae NOTRY=true Committed: https://crrev.com/71b669adca7b10a7fcecec151ec8b1c66a4c90a0 Cr-Commit-Position: refs/heads/master@{#439274} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/71b669adca7b10a7fcecec151ec8b1c66a4c90a0 Cr-Commit-Position: refs/heads/master@{#439274} |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
