Le 11/09/2009 11:42, Christian Thalinger a écrit :
> Rémi Forax wrote:
>
>> Before trying to apply the inlining patch, is there a reason why it is
>> guarded by testable ?
>>
> Not really, it's working quite good. But we still have these GC issues,
> which also apply to the compiler patch... :-/
>
> -- Christian
>
Christian,
the inlining patch currently doesn't work :(
in methodHandleWalk.hpp (line 322) :
int cpool_oop_reference_put(int tag, int first_index, int second_index) {
if (index == 0) return 0; // this line doesn't compile
ConstantValue* cv = new ConstantValue(tag, first_index, second_index);
return _constants.append(cv);
}
I think it should be:
int cpool_oop_reference_put(int tag, int first_index, int second_index) {
if (first_index == 0 && second_index == 0) return 0;
ConstantValue* cv = new ConstantValue(tag, first_index, second_index);
return _constants.append(cv);
}
Rémi
_______________________________________________
mlvm-dev mailing list
mlvm-dev@openjdk.java.net
http://mail.openjdk.java.net/mailman/listinfo/mlvm-dev
