AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[Turbofan] Intrinsic lowering of ToString(<str>) should provide <str>

Browse Source 
BUG=v8:6103, chromium:700294
R=bmeurer@chromium.org

Review-Url: https://codereview.chromium.org/2756463002
Cr-Commit-Position: refs/heads/master@{#43838}
This commit is contained in:
mvstanton 2017-03-15 11:35:16 -07:00 committed by Commit bot
parent 339bb9ff46
commit 7b76d52083
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/compiler/js-intrinsic-lowering.cc
View File

@ -298,6 +298,12 @@ Reduction JSIntrinsicLowering::ReduceToObject(Node* node) {
Reduction JSIntrinsicLowering::ReduceToString(Node* node) {
// ToString is unnecessary if the input is a string.
HeapObjectMatcher m(NodeProperties::GetValueInput(node, 0));
if (m.HasValue() && m.Value()->IsString()) {
ReplaceWithValue(node, m.node());
return Replace(m.node());
}
NodeProperties::ChangeOp(node, javascript()->ToString());
return Changed(node);
}