Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(531)

Side by Side Diff: src/checks.h

Issue 15302004: Convert ToBooleanStub to a HydrogenStub. Currently just using HBranch, which is still fully impleme… (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 247 matching lines...) Expand 10 before | Expand all | Expand 10 after
258 // generates code in debug builds. 258 // generates code in debug builds.
259 #ifdef DEBUG 259 #ifdef DEBUG
260 #define ASSERT_RESULT(expr) CHECK(expr) 260 #define ASSERT_RESULT(expr) CHECK(expr)
261 #define ASSERT(condition) CHECK(condition) 261 #define ASSERT(condition) CHECK(condition)
262 #define ASSERT_EQ(v1, v2) CHECK_EQ(v1, v2) 262 #define ASSERT_EQ(v1, v2) CHECK_EQ(v1, v2)
263 #define ASSERT_NE(v1, v2) CHECK_NE(v1, v2) 263 #define ASSERT_NE(v1, v2) CHECK_NE(v1, v2)
264 #define ASSERT_GE(v1, v2) CHECK_GE(v1, v2) 264 #define ASSERT_GE(v1, v2) CHECK_GE(v1, v2)
265 #define ASSERT_LT(v1, v2) CHECK_LT(v1, v2) 265 #define ASSERT_LT(v1, v2) CHECK_LT(v1, v2)
266 #define ASSERT_LE(v1, v2) CHECK_LE(v1, v2) 266 #define ASSERT_LE(v1, v2) CHECK_LE(v1, v2)
267 #define SLOW_ASSERT(condition) CHECK(!FLAG_enable_slow_asserts || (condition)) 267 #define SLOW_ASSERT(condition) CHECK(!FLAG_enable_slow_asserts || (condition))
268 #define FOR_ASSERT(statement) statement
268 #else 269 #else
269 #define ASSERT_RESULT(expr) (expr) 270 #define ASSERT_RESULT(expr) (expr)
270 #define ASSERT(condition) ((void) 0) 271 #define ASSERT(condition) ((void) 0)
271 #define ASSERT_EQ(v1, v2) ((void) 0) 272 #define ASSERT_EQ(v1, v2) ((void) 0)
272 #define ASSERT_NE(v1, v2) ((void) 0) 273 #define ASSERT_NE(v1, v2) ((void) 0)
273 #define ASSERT_GE(v1, v2) ((void) 0) 274 #define ASSERT_GE(v1, v2) ((void) 0)
274 #define ASSERT_LT(v1, v2) ((void) 0) 275 #define ASSERT_LT(v1, v2) ((void) 0)
275 #define ASSERT_LE(v1, v2) ((void) 0) 276 #define ASSERT_LE(v1, v2) ((void) 0)
276 #define SLOW_ASSERT(condition) ((void) 0) 277 #define SLOW_ASSERT(condition) ((void) 0)
278 #define FOR_ASSERT(statement) ((void) 0)
Toon Verwaest 2013/05/28 09:33:32 What about just putting the statement in #ifdef DE
277 #endif 279 #endif
278 // Static asserts has no impact on runtime performance, so they can be 280 // Static asserts has no impact on runtime performance, so they can be
279 // safely enabled in release mode. Moreover, the ((void) 0) expression 281 // safely enabled in release mode. Moreover, the ((void) 0) expression
280 // obeys different syntax rules than typedef's, e.g. it can't appear 282 // obeys different syntax rules than typedef's, e.g. it can't appear
281 // inside class declaration, this leads to inconsistency between debug 283 // inside class declaration, this leads to inconsistency between debug
282 // and release compilation modes behavior. 284 // and release compilation modes behavior.
283 #define STATIC_ASSERT(test) STATIC_CHECK(test) 285 #define STATIC_ASSERT(test) STATIC_CHECK(test)
284 286
285 #define ASSERT_NOT_NULL(p) ASSERT_NE(NULL, p) 287 #define ASSERT_NOT_NULL(p) ASSERT_NE(NULL, p)
286 288
287 // "Extra checks" are lightweight checks that are enabled in some release 289 // "Extra checks" are lightweight checks that are enabled in some release
288 // builds. 290 // builds.
289 #ifdef ENABLE_EXTRA_CHECKS 291 #ifdef ENABLE_EXTRA_CHECKS
290 #define EXTRA_CHECK(condition) CHECK(condition) 292 #define EXTRA_CHECK(condition) CHECK(condition)
291 #else 293 #else
292 #define EXTRA_CHECK(condition) ((void) 0) 294 #define EXTRA_CHECK(condition) ((void) 0)
293 #endif 295 #endif
294 296
295 #endif // V8_CHECKS_H_ 297 #endif // V8_CHECKS_H_
OLDNEW
« no previous file with comments | « src/arm/lithium-codegen-arm.cc ('k') | src/code-stubs.h » ('j') | src/code-stubs.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698