[aarch64-port-dev ] RFR: 8153837 : aarch64: handle special cases for MaxINode & MinINode

Felix Yang felix.yang at linaro.org
Mon Apr 11 12:48:02 UTC 2016


    Yes, I haven't looked into the details of the loop transformation code
and find it hard to produce a test case at least for now.
    And it seems to me necessary to do so in order to produce a good JTReg
test case for the patch.
    But I am not quite sure if I can come up with a test case which covers
all the templates added.
    Any suggestions?


On 11 April 2016 at 19:56, Andrew Haley <aph at redhat.com> wrote:

> Hi,
> On 04/11/2016 12:47 PM, Felix Yang wrote:
> >
> >     Thanks for reviewing the patch.
> >     I find that the cases the patch tries to catch here are the
> >     result of loop transformations.
> >     And it's hard to produce a test case to for it simply using the
> >     Math.min/max API.  (Seems C2 will not create a MaxINode/MinINode
> >     for a call for these APIs) But I do noticed some JTReg hotspot
> >     test cases that already generates the pattern.
> >
> >     So the patch got tested for the most part and is not causing us
> trouble.
> Sure, but that doesn't necessarily give us test coverage of the
> changes you've made.  Is the problem that you don't know how to write
> test cases to cover your changes?
> Andrew.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160411/9030b0c4/attachment.html>

More information about the hotspot-compiler-dev mailing list