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

Side by Side Diff: src/runtime.h

Issue 6902144: Handle join of sparse arrays with non-empty separator more efficiently. (Closed)
Patch Set: Add test for empty separator. Fix brainfart. Created 9 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
« no previous file with comments | « src/array.js ('k') | src/runtime.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 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 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 F(NumberSub, 2, 1) \ 125 F(NumberSub, 2, 1) \
126 F(NumberMul, 2, 1) \ 126 F(NumberMul, 2, 1) \
127 F(NumberDiv, 2, 1) \ 127 F(NumberDiv, 2, 1) \
128 F(NumberMod, 2, 1) \ 128 F(NumberMod, 2, 1) \
129 F(NumberUnaryMinus, 1, 1) \ 129 F(NumberUnaryMinus, 1, 1) \
130 F(NumberAlloc, 0, 1) \ 130 F(NumberAlloc, 0, 1) \
131 \ 131 \
132 F(StringAdd, 2, 1) \ 132 F(StringAdd, 2, 1) \
133 F(StringBuilderConcat, 3, 1) \ 133 F(StringBuilderConcat, 3, 1) \
134 F(StringBuilderJoin, 3, 1) \ 134 F(StringBuilderJoin, 3, 1) \
135 F(SparseJoinWithSeparator, 3, 1) \
135 \ 136 \
136 /* Bit operations */ \ 137 /* Bit operations */ \
137 F(NumberOr, 2, 1) \ 138 F(NumberOr, 2, 1) \
138 F(NumberAnd, 2, 1) \ 139 F(NumberAnd, 2, 1) \
139 F(NumberXor, 2, 1) \ 140 F(NumberXor, 2, 1) \
140 F(NumberNot, 1, 1) \ 141 F(NumberNot, 1, 1) \
141 \ 142 \
142 F(NumberShl, 2, 1) \ 143 F(NumberShl, 2, 1) \
143 F(NumberShr, 2, 1) \ 144 F(NumberShr, 2, 1) \
144 F(NumberSar, 2, 1) \ 145 F(NumberSar, 2, 1) \
(...skipping 490 matching lines...) Expand 10 before | Expand all | Expand 10 after
635 Handle<Script> script, 636 Handle<Script> script,
636 int position); 637 int position);
637 638
638 // Helper functions used stubs. 639 // Helper functions used stubs.
639 static void PerformGC(Object* result); 640 static void PerformGC(Object* result);
640 }; 641 };
641 642
642 } } // namespace v8::internal 643 } } // namespace v8::internal
643 644
644 #endif // V8_RUNTIME_H_ 645 #endif // V8_RUNTIME_H_
OLDNEW
« no previous file with comments | « src/array.js ('k') | src/runtime.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698