RFR: 8278079: C2: expand_dtrace_alloc_probe doesn't take effect in macro.cpp

Denghui Dong ddong at openjdk.java.net
Thu Dec 2 13:25:28 UTC 2021


On Wed, 1 Dec 2021 15:41:07 GMT, Denghui Dong <ddong at openjdk.org> wrote:

> Hi,
> 
> Could I have a review of this small fix that makes expand_dtrace_alloc_probe take effect?
> 
> Thanks,
> Denghui

src/hotspot/share/opto/macro.cpp line 1633:

> 1631: void PhaseMacroExpand::expand_dtrace_alloc_probe(AllocateNode* alloc, Node* oop,
> 1632:                                                 Node*& ctrl, Node*& rawmem) {
> 1633:   if (C->env()->dtrace_alloc_probes()) {

IIUC, this probe is related to object allocation, so it should be expanded when ciEnv::dtrace_alloc_probes() returns true, just like the implementation in C1(see C1_MacroAssembler::initialize_object().

-------------

PR: https://git.openjdk.java.net/jdk/pull/6639


More information about the hotspot-compiler-dev mailing list