Mercurial > hg > truffle
view src/share/tools/MakeDeps/UnixPlatform.java @ 1047:beb8f45ee9f0
6889740: G1: OpenDS fails with "unhandled exception in compiled code"
Summary: Incorrect code was being generated for the store operation in the null case of the aastore bytecode template. The bad code was generated by the store_heap_oop routine which takes a Register as its second argument. Passing NULL_WORD (0) as the second argument causes the value to be converted to Register(0), which is rax. Thus the generated store was "mov (dst), $rax" instead of "mov (dst), $0x0". Changed calls to store_heap_oop that pass NULL_WORD as the second argument to a new routine store_heap_oop_null.
Reviewed-by: kvn, twisti
author | johnc |
---|---|
date | Thu, 29 Oct 2009 09:42:26 -0700 |
parents | a61af66fc99e |
children | c18cbe5936b8 |
line wrap: on
line source
/* * Copyright 1999-2005 Sun Microsystems, Inc. All Rights Reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License version 2 only, as * published by the Free Software Foundation. * * This code is distributed in the hope that it will be useful, but WITHOUT * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License * version 2 for more details (a copy is included in the LICENSE file that * accompanied this code). * * You should have received a copy of the GNU General Public License version * 2 along with this work; if not, write to the Free Software Foundation, * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. * * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara, * CA 95054 USA or visit www.sun.com if you need additional information or * have any questions. * */ public class UnixPlatform extends Platform { public void setupFileTemplates() { inclFileTemplate = new FileName(this, "incls/", "_", "", ".incl", "", "" ); giFileTemplate = new FileName(this, "incls/", "", "_precompiled", ".incl", "", "" ); gdFileTemplate = new FileName(this, "", "", "Dependencies", "", "", "" ); } private static String[] suffixes = { ".cpp", ".c", ".s" }; public String[] outerSuffixes() { return suffixes; } public String objFileSuffix() { return ".o"; } public String asmFileSuffix() { return ".i"; } public String dependentPrefix() { return ""; } /** Do not change this; unless you fix things so precompiled header files get translated into make dependencies. - Ungar */ public int defaultGrandIncludeThreshold() { if (System.getProperty("USE_PRECOMPILED_HEADER") != null) return 30; else return 1 << 30; } /** For Unix make, include the dependencies for precompiled header files. */ public boolean includeGIDependencies() { return false; } /** Should C/C++ source file be dependent on a file included into the grand-include file. On Unix with precompiled headers we don't want each file to be dependent on grand-include file. Instead each C/C++ source file is depended on each own set of files, and recompiled only when files from this set are changed. */ public boolean writeDependenciesOnHFilesFromGI() { return System.getProperty("USE_PRECOMPILED_HEADER") != null; } }