[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-hotspot-compiler-dev
Subject: Re: RFR(S): 8148101: [JVMCI] Make CallingConvention.Type extensible
From: Christian Thalinger <christian.thalinger () oracle ! com>
Date: 2016-01-22 23:00:52
Message-ID: 31EB408D-FAF7-44CF-BBF3-45CA09960FED () oracle ! com
[Download RAW message or body]
http://cr.openjdk.java.net/~never/8148101/webrev/
Looks good.
> On Jan 22, 2016, at 12:25 PM, Tom Rodriguez <tom.rodriguez@oracle.com> wrote:
>
> https://bugs.openjdk.java.net/browse/JDK-8148101 \
> <https://bugs.openjdk.java.net/browse/JDK-8148101>
>
> CallingConvention.Type currently fixes the set of types for all possible backend. \
> It's should be abstracted so that it can be more easily extended. The unused \
> stackOnly parameter was removed at the same time.
> tom
[Attachment #3 (unknown)]
<html><head><meta http-equiv="Content-Type" content="text/html \
charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: \
space; -webkit-line-break: after-white-space;" class=""><a \
href="http://cr.openjdk.java.net/~never/8148101/webrev/" \
class="">http://cr.openjdk.java.net/~never/8148101/webrev/</a><div class=""><br \
class=""></div><div class="">Looks good.</div><div class=""><br \
class=""><div><blockquote type="cite" class=""><div class="">On Jan 22, 2016, at \
12:25 PM, Tom Rodriguez <tom.rodriguez@oracle.com> wrote:</div><br \
class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" \
content="text/html charset=us-ascii" class=""><div style="word-wrap: break-word; \
-webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><a \
href="https://bugs.openjdk.java.net/browse/JDK-8148101" \
class="">https://bugs.openjdk.java.net/browse/JDK-8148101</a><div class=""><br \
class=""></div><div class=""><br class=""></div><div class="">CallingConvention.Type \
currently fixes the set of types for all possible backend. It's should \
be abstracted so that it can be more easily extended. The unused stackOnly \
parameter was removed at the same time.</div><div class=""><br class=""></div><div \
class="">tom</div></div></div></blockquote></div><br class=""></div></body></html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic