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

Unified Diff: tests/compiler/dart2js/cps_ir/input/optimize_indexers.dart

Issue 2246623002: Delete CPS IR (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: tests/compiler/dart2js/cps_ir/input/optimize_indexers.dart
diff --git a/tests/compiler/dart2js/cps_ir/input/optimize_indexers.dart b/tests/compiler/dart2js/cps_ir/input/optimize_indexers.dart
deleted file mode 100644
index 8150653aaeee8f78bd7ba70fc25d09f78a34b7c1..0000000000000000000000000000000000000000
--- a/tests/compiler/dart2js/cps_ir/input/optimize_indexers.dart
+++ /dev/null
@@ -1,35 +0,0 @@
-// Method to test: function(test)
-import 'package:expect/expect.dart';
-
-// This example illustrates a case we wish to do better in terms of inlining and
-// code generation.
-//
-// Naively this function would be compiled without inlining Wrapper.[],
-// JSArray.[] and Wrapper.[]= because:
-// JSArray.[] is too big (14 nodes)
-// Wrapper.[] is too big if we force inlining of JSArray (15 nodes)
-// Wrapper.[]= is even bigger (46 nodes)
-//
-// We now do specialization of [] and []= by adding guards and injecting builtin
-// operators. This made it possible to inline []. We still don't see []= inlined
-// yet, that might require that we improve the inlining counting heuristics a
-// bit.
-@NoInline()
-test(data, x) {
- data[x + 1] = data[x];
-}
-
-main() {
- var wrapper = new Wrapper();
- wrapper[33] = wrapper[1]; // make Wrapper.[]= and [] used more than once.
- print(test(new Wrapper(), int.parse('2')));
-}
-
-class Wrapper {
- final List arr = <bool>[true, false, false, true];
- operator[](int i) => this.arr[i];
- operator[]=(int i, v) {
- if (i > arr.length - 1) arr.length = i + 1;
- return arr[i] = v;
- }
-}
« no previous file with comments | « tests/compiler/dart2js/cps_ir/input/operators_8.dart ('k') | tests/compiler/dart2js/cps_ir/input/redundant_condition.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698