RFR: 8261125: Move VM_Operation to vmOperation.hpp [v2]
Ioi Lam
iklam at openjdk.java.net
Thu Feb 11 03:59:37 UTC 2021
On Fri, 5 Feb 2021 17:34:06 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> I'm fine with leaving vmOperations.hpp and vmOperation.hpp. It's not a big deal. commonVMOperations.hpp - too much noise!
> I agree with David. interfaceSupport.inline.hpp imports a lot of things so importing vmOperations.hpp is not a big deal. vmOperations.hpp imports #include "runtime/threadSMR.hpp" otherwise it has all the same imports as interfaceSupport.inline.hpp anyway.
> All these files are going to increase compilation time too.
> I stand by my check mark above!
OK, since no one is fond of further splitting the headers, and no one has vetoed the vmOperation.hpp name, I'll keep everything as originally proposed. Will do more testing and integrate.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2398
More information about the shenandoah-dev
mailing list