Implement super read.

Implement super read.

Committed: https://code.google.com/p/dart/source/detail?r=8140

[kasperl, 2012-05-31 05:30:08]

LGTM with a test case.

[ahe, 2012-05-31 05:33:54]

On 2012/05/31 05:30:08, kasperl wrote:
> LGTM with a test case.

I expect that I'll fix some existing failing tests.

[kasperl, 2012-05-31 05:35:17]

LGTM with a status file update? :-)

[ahe, 2012-05-31 05:35:40]

On 2012/05/31 05:33:54, ahe wrote:
> On 2012/05/31 05:30:08, kasperl wrote:
> > LGTM with a test case.
> 
> I expect that I'll fix some existing failing tests.

Indeed:

FAILED: dart2js-d8 release_ia32 language/super_field_test
Expected: fail 
Actual: pass

And three other tests so far.

[ahe, 2012-05-31 05:50:07]

Updated status file.

[ngeoffray, 2012-05-31 07:00:57]

LGTM! Will you also do super write?

https://chromiumcodereview.appspot.com/10442107/diff/5001/dart/lib/compiler/i...
File dart/lib/compiler/implementation/ssa/codegen.dart (right):

https://chromiumcodereview.appspot.com/10442107/diff/5001/dart/lib/compiler/i...
dart/lib/compiler/implementation/ssa/codegen.dart:1564:
buffer.add('$className.prototype.$methodName.call(this)');
How could that code have ever worked before? Is that the reason why
mint_maker_rpc_test was failing?

[ahe, 2012-05-31 07:24:35]

Thank you, Kasper and Nicolas.

I'll also do super write.

https://chromiumcodereview.appspot.com/10442107/diff/5001/dart/lib/compiler/i...
File dart/lib/compiler/implementation/ssa/codegen.dart (right):

https://chromiumcodereview.appspot.com/10442107/diff/5001/dart/lib/compiler/i...
dart/lib/compiler/implementation/ssa/codegen.dart:1564:
buffer.add('$className.prototype.$methodName.call(this)');
On 2012/05/31 07:00:57, ngeoffray wrote:
> How could that code have ever worked before? Is that the reason why
> mint_maker_rpc_test was failing?

I think so.

Actually,  I think I need to change it to call visitArguments.