)]}' { "commit": "c47f7936d0ec1a9c6c0a5fad1787c6acd7c25fc9", "tree": "e1777f57e0667507c3c6c37c099f798facc0dfd5", "parents": [ "51011b4a262046a3d9a2eb5690e54af80377f15a" ], "author": { "name": "David Benjamin", "email": "davidben@google.com", "time": "Tue Jan 22 19:49:35 2019 -0600" }, "committer": { "name": "CQ bot account: commit-bot@chromium.org", "email": "commit-bot@chromium.org", "time": "Wed Jan 23 17:21:56 2019 +0000" }, "message": "Add a RelWithAsserts build configuration.\n\nOn our bots, debug unit tests take around 2.5x as long to complete as\nrelease tests on Linux, 3x as long on macOS, and 6x as long on Windows.\nOur tests are fast, so this does not particularly matter, but SDE\ninflates a 13 second test run to 8 minutes. On Windows (MSVC), where we\ndon\u0027t but would like to test with SDE, the difference between optimized\nand unoptimized is even larger, and test runs are slower in general.\n\nThis suggests running SDE tests in release mode. Release mode tests,\nhowever, are less effective because they do not include asserts. Thus,\nadd a RelWithAsserts option.\n\n(Chromium does something similar. I believe most of the test-running\nconfigurations on the critical path run is_debug \u003d false and\ndcheck_always_on \u003d true.)\n\nChange-Id: I273dd86ab8ea039f34eca431483827c87dc5c461\nReviewed-on: https://boringssl-review.googlesource.com/c/34464\nCommit-Queue: Adam Langley \u003cagl@google.com\u003e\nReviewed-by: Adam Langley \u003cagl@google.com\u003e\n", "tree_diff": [ { "type": "modify", "old_id": "de92237c4c93ee3b1ba38b1c2c4d8e5b5567a647", "old_mode": 33188, "old_path": "CMakeLists.txt", "new_id": "fd3532664517f546e7cd3c32c2e74db17d84e00b", "new_mode": 33188, "new_path": "CMakeLists.txt" } ] }