Skip to content

Commit

Permalink
Get correct TR_ALLOC definition in jitbuilder files
Browse files Browse the repository at this point in the history
Due to header file ordering, the wrong (empty) declaration of TR_ALLOC
is used in OMRIlBuilder.cpp and OMRVirtualMachineOperandStack.cpp.
Fix that by directly including env/TRMemory.hpp early.

Signed-off-by: Mark Stoodley <mstoodle@ca.ibm.com>
  • Loading branch information
mstoodle committed Aug 1, 2018
1 parent 6fa7636 commit bdea4f5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions compiler/ilgen/OMRIlBuilder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
* SPDX-License-Identifier: EPL-2.0 OR Apache-2.0 OR GPL-2.0 WITH Classpath-exception-2.0 OR LicenseRef-GPL-2.0 WITH Assembly-exception
*******************************************************************************/

#include "env/TRMemory.hpp" // must precede IlBuilder.hpp to get TR_ALLOC
#include "ilgen/IlBuilder.hpp"

#include <stdint.h>
Expand Down
2 changes: 2 additions & 0 deletions compiler/ilgen/OMRVirtualMachineOperandStack.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@
* SPDX-License-Identifier: EPL-2.0 OR Apache-2.0 OR GPL-2.0 WITH Classpath-exception-2.0 OR LicenseRef-GPL-2.0 WITH Assembly-exception
*******************************************************************************/

#include "env/TRMemory.hpp" // include first to get correct TR_ALLOC definition
#include "ilgen/VirtualMachineOperandStack.hpp"

#include "ilgen/VirtualMachineRegister.hpp"
#include "compile/Compilation.hpp"
#include "il/SymbolReference.hpp"
Expand Down

0 comments on commit bdea4f5

Please sign in to comment.