Collect getters in a FunctionSet while resolving.

tests/compiler/dart2js/gvn_dynamic_field_get_test.dart

Index: tests/compiler/dart2js/gvn_dynamic_field_get_test.dart
===================================================================
--- tests/compiler/dart2js/gvn_dynamic_field_get_test.dart	(revision 0)
+++ tests/compiler/dart2js/gvn_dynamic_field_get_test.dart	(revision 0)
@@ -0,0 +1,27 @@
+// Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+// Test that dart2js gvns dynamic getters that don't have side
+// effects.
+
+#import('compiler_helper.dart');
+
+const String TEST = @"""
+class A {
+  var foo;
+  bar(a) {
+    return a.foo + a.foo;
+  }
+}
+
+main() {
+  new A().bar(new Object());
+}
+""";
+
+main() {
+  String generated = compileAll(TEST);
+  RegExp regexp = const RegExp(@"get\$foo");
+  Iterator matches = regexp.allMatches(generated).iterator();
+  checkNumberOfMatches(matches, 1);

[Søren Gjesse, 2012/09/11 08:06:01]

Shouldn't this test also contain a lookup of the getter "foo" in the World
getters function set?

[ngeoffray, 2012/09/11 08:51:08]

Done.

+}