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

Unified Diff: src/builtins/builtins-string.cc

Issue 2489743002: [stubs] Ensure CSA_ASSERT and CSA_SLOW_ASSERT do not produce unused instructions in release mode. (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: src/builtins/builtins-string.cc
diff --git a/src/builtins/builtins-string.cc b/src/builtins/builtins-string.cc
index 7828dd231f1405874ee451211950f874c1562083..f7419af743da38c9e179b1e0d641a46aec097aa1 100644
--- a/src/builtins/builtins-string.cc
+++ b/src/builtins/builtins-string.cc
@@ -1065,8 +1065,8 @@ void Builtins::Generate_StringPrototypeSubstr(CodeStubAssembler* a) {
// two cases according to the spec: if it is negative, "" is returned; if
// it is positive, then length is set to {string_length} - {start}.
- a->Assert(a->WordEqual(a->LoadMap(var_length.value()),
- a->HeapNumberMapConstant()));
+ a->CSA_ASSERT(a->WordEqual(a->LoadMap(var_length.value()),
+ a->HeapNumberMapConstant()));
Label if_isnegative(a), if_ispositive(a);
Node* const float_zero = a->Float64Constant(0.);
@@ -1135,7 +1135,8 @@ compiler::Node* ToSmiBetweenZeroAnd(CodeStubAssembler* a,
a->Bind(&if_isnotsmi);
{
// {value} is a heap number - in this case, it is definitely out of bounds.
- a->Assert(a->WordEqual(a->LoadMap(value_int), a->HeapNumberMapConstant()));
+ a->CSA_ASSERT(
+ a->WordEqual(a->LoadMap(value_int), a->HeapNumberMapConstant()));
Node* const float_zero = a->Float64Constant(0.);
Node* const smi_zero = a->SmiConstant(Smi::kZero);
@@ -1367,13 +1368,13 @@ compiler::Node* LoadSurrogatePairInternal(CodeStubAssembler* assembler,
Node* trail = var_trail.value();
#ifdef ENABLE_SLOW_DCHECKS
// Check that this path is only taken if a surrogate pair is found
- assembler->Assert(assembler->Uint32GreaterThanOrEqual(
+ assembler->CSA_ASSERT(assembler->Uint32GreaterThanOrEqual(
lead, assembler->Int32Constant(0xD800)));
- assembler->Assert(
+ assembler->CSA_ASSERT(
assembler->Uint32LessThan(lead, assembler->Int32Constant(0xDC00)));
- assembler->Assert(assembler->Uint32GreaterThanOrEqual(
+ assembler->CSA_ASSERT(assembler->Uint32GreaterThanOrEqual(
trail, assembler->Int32Constant(0xDC00)));
- assembler->Assert(
+ assembler->CSA_ASSERT(
assembler->Uint32LessThan(trail, assembler->Int32Constant(0xE000)));
#endif

Powered by Google App Engine
This is Rietveld 408576698