| commit | 9393e8e433ae9255eec4de65671364d363ca6657 | [log] [tgz] |
|---|---|---|
| author | Jim Pollock <[email protected]> | Mon May 03 23:52:43 2021 |
| committer | Commit Bot <[email protected]> | Mon May 10 10:13:34 2021 |
| tree | 7779482244618beff11f321dda278940d677e0e2 | |
| parent | 6e41879e12f1697a02474be2def14ba81b79e36a [diff] |
nn: enable NDEBUG, but unset for tests This CL allows the NDEBUG flag to be set for the core libraries, which otherwise results in a linking error when using refptr due to the destructor of RefCountedThreadSafeBase being defined differently based on the NDEBUG flag. See b/174605330#comment13 for details. We still require NDEBUG to be unset for tests, so this CL will unset the define for the test targets, otherwise tests will not build or pass correctly. BUG=b:186996014 TEST=FEATURES=test USE=-cros-debug emerge-volteer aosp-frameworks-ml-nn Change-Id: Icfc69dcd08f96577cc3ca30b0c05df556bfb0550 Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/frameworks/ml/+/2870225 Tested-by: Jim Pollock <[email protected]> Auto-Submit: Jim Pollock <[email protected]> Commit-Queue: Jim Pollock <[email protected]> Reviewed-by: Stuart Langley <[email protected]>