Commit 00215f30 authored by Bill Budge's avatar Bill Budge Committed by Commit Bot

[torque] Clean up usage of NonNumberToNumber

Bug: v8:9891

Change-Id: I601dd2a4e1c7d1e3818a55b92e83ce6b072c7ad0
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2238486Reviewed-by: 's avatarJakob Gruber <jgruber@chromium.org>
Commit-Queue: Bill Budge <bbudge@chromium.org>
Cr-Commit-Position: refs/heads/master@{#68306}
parent a4c9b582
...@@ -474,7 +474,7 @@ extern macro Print(Object); ...@@ -474,7 +474,7 @@ extern macro Print(Object);
extern macro DebugBreak(); extern macro DebugBreak();
// ES6 7.1.4 ToInteger ( argument ) // ES6 7.1.4 ToInteger ( argument )
transitioning macro ToIntegerImpl(implicit context: Context)(input: Object): transitioning macro ToIntegerImpl(implicit context: Context)(input: JSAny):
Number { Number {
let input = input; let input = input;
...@@ -493,28 +493,28 @@ transitioning macro ToIntegerImpl(implicit context: Context)(input: Object): ...@@ -493,28 +493,28 @@ transitioning macro ToIntegerImpl(implicit context: Context)(input: Object):
assert(IsNumberNormalized(result)); assert(IsNumberNormalized(result));
return result; return result;
} }
case (ho: HeapObject): { case (a: JSAnyNotNumber): {
input = math::NonNumberToNumber(ho); input = conversion::NonNumberToNumber(a);
} }
} }
} }
unreachable; unreachable;
} }
transitioning builtin ToInteger(implicit context: Context)(input: Object): transitioning builtin ToInteger(implicit context: Context)(input: JSAny):
Number { Number {
return ToIntegerImpl(input); return ToIntegerImpl(input);
} }
@export @export
transitioning macro ToInteger_Inline(implicit context: Context)(input: Object): transitioning macro ToInteger_Inline(implicit context: Context)(input: JSAny):
Number { Number {
typeswitch (input) { typeswitch (input) {
case (s: Smi): { case (s: Smi): {
return s; return s;
} }
case (ho: HeapObject): { case (JSAny): {
return ToInteger(ho); return ToInteger(input);
} }
} }
} }
......
...@@ -4,9 +4,6 @@ ...@@ -4,9 +4,6 @@
namespace math { namespace math {
extern transitioning builtin
NonNumberToNumber(implicit context: Context)(HeapObject): Number;
transitioning macro ReduceToSmiOrFloat64(implicit context: Context)(x: JSAny): transitioning macro ReduceToSmiOrFloat64(implicit context: Context)(x: JSAny):
never never
labels SmiResult(Smi), Float64Result(float64) { labels SmiResult(Smi), Float64Result(float64) {
...@@ -20,7 +17,7 @@ transitioning macro ReduceToSmiOrFloat64(implicit context: Context)(x: JSAny): ...@@ -20,7 +17,7 @@ transitioning macro ReduceToSmiOrFloat64(implicit context: Context)(x: JSAny):
goto Float64Result(Convert<float64>(h)); goto Float64Result(Convert<float64>(h));
} }
case (a: JSAnyNotNumber): { case (a: JSAnyNotNumber): {
x1 = NonNumberToNumber(a); x1 = conversion::NonNumberToNumber(a);
} }
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment