RFR(S): 8165056: move JIT Compiler related files from runtime/ to compiler/ directory

[Liu Xin]

Hi, Coleen and Vladimir,

Indeed, some files don't depend on the header compilationPolicy.hpp, so I
remove them. In particular, jni.cpp actually depends on jvmciCompiler.hpp
instead of compilationPolicy.hpp.

I did extra hg massage to make it clear.
https://cr.openjdk.java.net/~xliu/8165056/v2/webrev/

I built it successfully and it's still running tests.

thanks,
--lx


On Wed, Oct 9, 2019 at 11:31 AM Vladimir Kozlov <vladimir.kozlov at oracle.com>
wrote:

> On 10/9/19 5:57 AM, coleen.phillimore at oracle.com wrote:
> >
> > Thank you for doing this!
> >
> > Can you check that compilationPolicy.hpp is actually needed to be
> > included from the non-compiler files?
>
> Yes, it would be nice to clean up not-needed #include.
>
> >
> > Did you use hg mv for compilationPolicy* ?
>
> Webrev show previous files location which I assume indication of `hg mv`
>
> Thanks,
> Vladimir
>
> >
> > Thanks,
> > Coleen
> >
> > On 10/9/19 5:12 AM, Liu Xin wrote:
> >> Hi, Reviewers,
> >>
> >> Could you review this webrev?
> >> https://cr.openjdk.java.net/~xliu/8165056/webrev/
> >> JBS: https://bugs.openjdk.java.net/browse/JDK-8165056
> >>
> >> The change is straightforward. Those 3 files below have been merged
> >> into compilationPolicy.hpp/cpp.
> >>
> >> simpleThresholdPolicy.cpp
> >> simpleThresholdPolicy.hpp
> >> simpleThresholdPolicy.inline.hpp
> >>
> >> I ran hotspot/compiler tests and it's clear.
> >>
> >> thanks,
> >> --lx
> >
>