make/windows/makefiles/compile.make

Fri, 20 Sep 2013 10:53:28 +0200

author
stefank
date
Fri, 20 Sep 2013 10:53:28 +0200
changeset 5769
2c022e432e10
parent 5417
33c52908bcdb
child 5842
febab3a8f203
child 5877
7638e35cabc6
permissions
-rw-r--r--

8024974: Incorrect use of GC_locker::is_active()
Summary: SymbolTable and StringTable can make calls to GC_locker::is_active() outside a safepoint. This isn't safe because the GC_locker active state (lock count) is only updated at a safepoint and only remains valid as long as _needs_gc is true. However, outside a safepoint_needs_gc can change to false at any time, which makes it impossible to do a correct call to is_active() in that context. In this case these calls can just be removed since the input argument to basic_add() should never be on the heap and so there's no need to check the GC_locker state. This change also adjusts the assert() in is_active() to makes sure all calls to this function are always done under a safepoint.
Reviewed-by: brutisso, dcubed
Contributed-by: per.liden@oracle.com

duke@435 1 #
dholmes@5417 2 # Copyright (c) 1997, 2013, Oracle and/or its affiliates. All rights reserved.
duke@435 3 # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
duke@435 4 #
duke@435 5 # This code is free software; you can redistribute it and/or modify it
duke@435 6 # under the terms of the GNU General Public License version 2 only, as
duke@435 7 # published by the Free Software Foundation.
duke@435 8 #
duke@435 9 # This code is distributed in the hope that it will be useful, but WITHOUT
duke@435 10 # ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
duke@435 11 # FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
duke@435 12 # version 2 for more details (a copy is included in the LICENSE file that
duke@435 13 # accompanied this code).
duke@435 14 #
duke@435 15 # You should have received a copy of the GNU General Public License version
duke@435 16 # 2 along with this work; if not, write to the Free Software Foundation,
duke@435 17 # Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
duke@435 18 #
trims@1907 19 # Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
trims@1907 20 # or visit www.oracle.com if you need additional information or have any
trims@1907 21 # questions.
duke@435 22 #
duke@435 23 #
duke@435 24
duke@435 25 # Generic compiler settings
erikj@3600 26 !if "x$(CXX)" == "x"
erikj@3518 27 CXX=cl.exe
erikj@3600 28 !endif
duke@435 29
erikj@3518 30 # CXX Flags: (these vary slightly from VC6->VS2003->VS2005 compilers)
duke@435 31 # /nologo Supress copyright message at every cl.exe startup
duke@435 32 # /W3 Warning level 3
duke@435 33 # /Zi Include debugging information
duke@435 34 # /WX Treat any warning error as a fatal error
tbell@862 35 # /MD Use dynamic multi-threaded runtime (msvcrt.dll or msvc*NN.dll)
duke@435 36 # /MTd Use static multi-threaded runtime debug versions
duke@435 37 # /O1 Optimize for size (/Os), skips /Oi
duke@435 38 # /O2 Optimize for speed (/Ot), adds /Oi to /O1
duke@435 39 # /Ox Old "all optimizations flag" for VC6 (in /O1)
duke@435 40 # /Oy Use frame pointer register as GP reg (in /Ox and /O1)
duke@435 41 # /GF Merge string constants and put in read-only memory (in /O1)
duke@435 42 # /Gy Func level link (in /O1, allows for link-time func ordering)
duke@435 43 # /Gs Inserts stack probes (in /O1)
duke@435 44 # /GS Inserts security stack checks in some functions (VS2005 default)
duke@435 45 # /Oi Use intrinsics (in /O2)
duke@435 46 # /Od Disable all optimizations
duke@435 47 #
duke@435 48 # NOTE: Normally following any of the above with a '-' will turn off that flag
sbohne@492 49 #
sbohne@492 50 # 6655385: For VS2003/2005 we now specify /Oy- (disable frame pointer
sbohne@492 51 # omission.) This has little to no effect on performance while vastly
sbohne@492 52 # improving the quality of crash log stack traces involving jvm.dll.
duke@435 53
duke@435 54 # These are always used in all compiles
sla@5046 55 CXX_FLAGS=$(EXTRA_CFLAGS) /nologo /W3 /WX
duke@435 56
dcubed@3724 57 # Let's add debug information when Full Debug Symbols is enabled
dcubed@3724 58 !if "$(ENABLE_FULL_DEBUG_SYMBOLS)" == "1"
erikj@3518 59 CXX_FLAGS=$(CXX_FLAGS) /Zi
dcubed@3724 60 !endif
duke@435 61
duke@435 62 # Based on BUILDARCH we add some flags and select the default compiler name
duke@435 63 !if "$(BUILDARCH)" == "ia64"
duke@435 64 MACHINE=IA64
duke@435 65 DEFAULT_COMPILER_NAME=VS2003
erikj@3518 66 CXX_FLAGS=$(CXX_FLAGS) /D "CC_INTERP" /D "_LP64" /D "IA64"
duke@435 67 !endif
duke@435 68
duke@435 69 !if "$(BUILDARCH)" == "amd64"
duke@435 70 MACHINE=AMD64
duke@435 71 DEFAULT_COMPILER_NAME=VS2005
erikj@3518 72 CXX_FLAGS=$(CXX_FLAGS) /D "_LP64" /D "AMD64"
duke@435 73 LP64=1
duke@435 74 !endif
duke@435 75
duke@435 76 !if "$(BUILDARCH)" == "i486"
duke@435 77 MACHINE=I386
duke@435 78 DEFAULT_COMPILER_NAME=VS2003
erikj@3518 79 CXX_FLAGS=$(CXX_FLAGS) /D "IA32"
duke@435 80 !endif
duke@435 81
duke@435 82 # Sanity check, this is the default if not amd64, ia64, or i486
duke@435 83 !ifndef DEFAULT_COMPILER_NAME
erikj@3518 84 CXX=ARCH_ERROR
duke@435 85 !endif
duke@435 86
erikj@3518 87 CXX_FLAGS=$(CXX_FLAGS) /D "WIN32" /D "_WINDOWS"
sla@2369 88 # Must specify this for sharedRuntimeTrig.cpp
erikj@3518 89 CXX_FLAGS=$(CXX_FLAGS) /D "VM_LITTLE_ENDIAN"
sla@2369 90
sla@2369 91 # Used for platform dispatching
erikj@3518 92 CXX_FLAGS=$(CXX_FLAGS) /D TARGET_OS_FAMILY_windows
erikj@3518 93 CXX_FLAGS=$(CXX_FLAGS) /D TARGET_ARCH_$(Platform_arch)
erikj@3518 94 CXX_FLAGS=$(CXX_FLAGS) /D TARGET_ARCH_MODEL_$(Platform_arch_model)
erikj@3518 95 CXX_FLAGS=$(CXX_FLAGS) /D TARGET_OS_ARCH_windows_$(Platform_arch)
erikj@3518 96 CXX_FLAGS=$(CXX_FLAGS) /D TARGET_OS_ARCH_MODEL_windows_$(Platform_arch_model)
erikj@3518 97 CXX_FLAGS=$(CXX_FLAGS) /D TARGET_COMPILER_visCPP
sla@2369 98
sla@2369 99
tbell@862 100 # MSC_VER is a 4 digit number that tells us what compiler is being used
tbell@862 101 # and is generated when the local.make file is created by build.make
tbell@862 102 # via the script get_msc_ver.sh
tbell@862 103 #
duke@435 104 # If MSC_VER is set, it overrides the above default setting.
duke@435 105 # But it should be set.
duke@435 106 # Possible values:
duke@435 107 # 1200 is for VC6
duke@435 108 # 1300 and 1310 is VS2003 or VC7
duke@435 109 # 1399 is our fake number for the VS2005 compiler that really isn't 1400
duke@435 110 # 1400 is for VS2005
tbell@862 111 # 1500 is for VS2008
prr@1840 112 # 1600 is for VS2010
dholmes@5417 113 # 1700 is for VS2012
duke@435 114 # Do not confuse this MSC_VER with the predefined macro _MSC_VER that the
duke@435 115 # compiler provides, when MSC_VER==1399, _MSC_VER will be 1400.
duke@435 116 # Normally they are the same, but a pre-release of the VS2005 compilers
duke@435 117 # in the Windows 64bit Platform SDK said it was 1400 when it was really
duke@435 118 # closer to VS2003 in terms of option spellings, so we use 1399 for that
duke@435 119 # 1400 version that really isn't 1400.
tbell@862 120 # See the file get_msc_ver.sh for more info.
duke@435 121 !if "x$(MSC_VER)" == "x"
duke@435 122 COMPILER_NAME=$(DEFAULT_COMPILER_NAME)
duke@435 123 !else
duke@435 124 !if "$(MSC_VER)" == "1200"
duke@435 125 COMPILER_NAME=VC6
duke@435 126 !endif
duke@435 127 !if "$(MSC_VER)" == "1300"
duke@435 128 COMPILER_NAME=VS2003
duke@435 129 !endif
duke@435 130 !if "$(MSC_VER)" == "1310"
duke@435 131 COMPILER_NAME=VS2003
duke@435 132 !endif
duke@435 133 !if "$(MSC_VER)" == "1399"
duke@435 134 # Compiler might say 1400, but if it's 14.00.30701, it isn't really VS2005
duke@435 135 COMPILER_NAME=VS2003
duke@435 136 !endif
duke@435 137 !if "$(MSC_VER)" == "1400"
duke@435 138 COMPILER_NAME=VS2005
duke@435 139 !endif
tbell@862 140 !if "$(MSC_VER)" == "1500"
tbell@862 141 COMPILER_NAME=VS2008
tbell@862 142 !endif
prr@1840 143 !if "$(MSC_VER)" == "1600"
prr@1840 144 COMPILER_NAME=VS2010
prr@1840 145 !endif
dholmes@5417 146 !if "$(MSC_VER)" == "1700"
dholmes@5417 147 COMPILER_NAME=VS2012
dholmes@5417 148 !endif
duke@435 149 !endif
duke@435 150
duke@435 151 # By default, we do not want to use the debug version of the msvcrt.dll file
duke@435 152 # but if MFC_DEBUG is defined in the environment it will be used.
duke@435 153 MS_RUNTIME_OPTION = /MD
duke@435 154 !if "$(MFC_DEBUG)" == "true"
duke@435 155 MS_RUNTIME_OPTION = /MTd /D "_DEBUG"
duke@435 156 !endif
duke@435 157
dholmes@5417 158 # VS2012 and later won't work with:
dholmes@5417 159 # /D _STATIC_CPPLIB /D _DISABLE_DEPRECATE_STATIC_CPPLIB
dholmes@5417 160 !if "$(MSC_VER)" < "1700"
duke@435 161 # Always add the _STATIC_CPPLIB flag
zgu@2396 162 STATIC_CPPLIB_OPTION = /D _STATIC_CPPLIB /D _DISABLE_DEPRECATE_STATIC_CPPLIB
duke@435 163 MS_RUNTIME_OPTION = $(MS_RUNTIME_OPTION) $(STATIC_CPPLIB_OPTION)
dholmes@5417 164 !endif
erikj@3518 165 CXX_FLAGS=$(CXX_FLAGS) $(MS_RUNTIME_OPTION)
duke@435 166
duke@435 167 # How /GX option is spelled
duke@435 168 GX_OPTION = /GX
duke@435 169
duke@435 170 # Optimization settings for various versions of the compilers and types of
duke@435 171 # builds. Three basic sets of settings: product, fastdebug, and debug.
erikj@3518 172 # These get added into CXX_FLAGS as needed by other makefiles.
duke@435 173 !if "$(COMPILER_NAME)" == "VC6"
duke@435 174 PRODUCT_OPT_OPTION = /Ox /Os /Gy /GF
duke@435 175 FASTDEBUG_OPT_OPTION = /Ox /Os /Gy /GF
duke@435 176 DEBUG_OPT_OPTION = /Od
duke@435 177 !endif
duke@435 178
duke@435 179 !if "$(COMPILER_NAME)" == "VS2003"
sbohne@492 180 PRODUCT_OPT_OPTION = /O2 /Oy-
sbohne@492 181 FASTDEBUG_OPT_OPTION = /O2 /Oy-
duke@435 182 DEBUG_OPT_OPTION = /Od
duke@435 183 !endif
duke@435 184
duke@435 185 !if "$(COMPILER_NAME)" == "VS2005"
sbohne@492 186 PRODUCT_OPT_OPTION = /O2 /Oy-
sbohne@492 187 FASTDEBUG_OPT_OPTION = /O2 /Oy-
duke@435 188 DEBUG_OPT_OPTION = /Od
duke@435 189 GX_OPTION = /EHsc
duke@435 190 # This VS2005 compiler has /GS as a default and requires bufferoverflowU.lib
duke@435 191 # on the link command line, otherwise we get missing __security_check_cookie
duke@435 192 # externals at link time. Even with /GS-, you need bufferoverflowU.lib.
duke@435 193 # NOTE: Currently we decided to not use /GS-
duke@435 194 BUFFEROVERFLOWLIB = bufferoverflowU.lib
erikj@3518 195 LD_FLAGS = /manifest $(LD_FLAGS) $(BUFFEROVERFLOWLIB)
tbell@862 196 # Manifest Tool - used in VS2005 and later to adjust manifests stored
tbell@862 197 # as resources inside build artifacts.
erikj@3600 198 !if "x$(MT)" == "x"
tbell@862 199 MT=mt.exe
tbell@862 200 !endif
erikj@3600 201 !endif
tbell@862 202
tbell@862 203 !if "$(COMPILER_NAME)" == "VS2008"
tbell@862 204 PRODUCT_OPT_OPTION = /O2 /Oy-
tbell@862 205 FASTDEBUG_OPT_OPTION = /O2 /Oy-
tbell@862 206 DEBUG_OPT_OPTION = /Od
tbell@862 207 GX_OPTION = /EHsc
erikj@3518 208 LD_FLAGS = /manifest $(LD_FLAGS)
tbell@862 209 # Manifest Tool - used in VS2005 and later to adjust manifests stored
tbell@862 210 # as resources inside build artifacts.
erikj@3600 211 !if "x$(MT)" == "x"
tbell@862 212 MT=mt.exe
duke@435 213 !endif
erikj@3600 214 !endif
duke@435 215
prr@1840 216 !if "$(COMPILER_NAME)" == "VS2010"
prr@1840 217 PRODUCT_OPT_OPTION = /O2 /Oy-
prr@1840 218 FASTDEBUG_OPT_OPTION = /O2 /Oy-
prr@1840 219 DEBUG_OPT_OPTION = /Od
prr@1840 220 GX_OPTION = /EHsc
erikj@3518 221 LD_FLAGS = /manifest $(LD_FLAGS)
prr@1840 222 # Manifest Tool - used in VS2005 and later to adjust manifests stored
prr@1840 223 # as resources inside build artifacts.
erikj@3600 224 !if "x$(MT)" == "x"
prr@1840 225 MT=mt.exe
erikj@3600 226 !endif
zgu@2590 227 !if "$(BUILDARCH)" == "i486"
erikj@3518 228 LD_FLAGS = /SAFESEH $(LD_FLAGS)
zgu@2590 229 !endif
prr@1840 230 !endif
prr@1840 231
dholmes@5417 232 !if "$(COMPILER_NAME)" == "VS2012"
dholmes@5417 233 PRODUCT_OPT_OPTION = /O2 /Oy-
dholmes@5417 234 FASTDEBUG_OPT_OPTION = /O2 /Oy-
dholmes@5417 235 DEBUG_OPT_OPTION = /Od
dholmes@5417 236 GX_OPTION = /EHsc
dholmes@5417 237 LD_FLAGS = /manifest $(LD_FLAGS)
dholmes@5417 238 # Manifest Tool - used in VS2005 and later to adjust manifests stored
dholmes@5417 239 # as resources inside build artifacts.
dholmes@5417 240 !if "x$(MT)" == "x"
dholmes@5417 241 MT=mt.exe
dholmes@5417 242 !endif
dholmes@5417 243 !if "$(BUILDARCH)" == "i486"
dholmes@5417 244 LD_FLAGS = /SAFESEH $(LD_FLAGS)
dholmes@5417 245 !endif
dholmes@5417 246 !endif
dholmes@5417 247
duke@435 248 # If NO_OPTIMIZATIONS is defined in the environment, turn everything off
duke@435 249 !ifdef NO_OPTIMIZATIONS
duke@435 250 PRODUCT_OPT_OPTION = $(DEBUG_OPT_OPTION)
duke@435 251 FASTDEBUG_OPT_OPTION = $(DEBUG_OPT_OPTION)
duke@435 252 !endif
duke@435 253
duke@435 254 # Generic linker settings
erikj@3600 255 !if "x$(LD)" == "x"
erikj@3518 256 LD=link.exe
erikj@3600 257 !endif
erikj@3518 258 LD_FLAGS= $(LD_FLAGS) kernel32.lib user32.lib gdi32.lib winspool.lib \
duke@435 259 comdlg32.lib advapi32.lib shell32.lib ole32.lib oleaut32.lib \
duke@435 260 uuid.lib Wsock32.lib winmm.lib /nologo /machine:$(MACHINE) /opt:REF \
dcubed@3724 261 /opt:ICF,8
dcubed@3724 262 !if "$(ENABLE_FULL_DEBUG_SYMBOLS)" == "1"
dcubed@3724 263 LD_FLAGS= $(LD_FLAGS) /map /debug
dcubed@3724 264 !endif
duke@435 265
zgu@3031 266
zgu@3031 267 !if $(MSC_VER) >= 1600
erikj@3518 268 LD_FLAGS= $(LD_FLAGS) psapi.lib
zgu@3031 269 !endif
zgu@3031 270
duke@435 271 # Resource compiler settings
erikj@3600 272 !if "x$(RC)" == "x"
duke@435 273 RC=rc.exe
erikj@3600 274 !endif
duke@435 275 RC_FLAGS=/D "HS_VER=$(HS_VER)" \
duke@435 276 /D "HS_DOTVER=$(HS_DOTVER)" \
duke@435 277 /D "HS_BUILD_ID=$(HS_BUILD_ID)" \
duke@435 278 /D "JDK_VER=$(JDK_VER)" \
duke@435 279 /D "JDK_DOTVER=$(JDK_DOTVER)" \
duke@435 280 /D "HS_COMPANY=$(HS_COMPANY)" \
duke@435 281 /D "HS_FILEDESC=$(HS_FILEDESC)" \
duke@435 282 /D "HS_COPYRIGHT=$(HS_COPYRIGHT)" \
duke@435 283 /D "HS_FNAME=$(HS_FNAME)" \
duke@435 284 /D "HS_INTERNAL_NAME=$(HS_INTERNAL_NAME)" \
duke@435 285 /D "HS_NAME=$(HS_NAME)"
duke@435 286
erikj@3518 287 # Need this to match the CXX_FLAGS settings
duke@435 288 !if "$(MFC_DEBUG)" == "true"
duke@435 289 RC_FLAGS = $(RC_FLAGS) /D "_DEBUG"
duke@435 290 !endif
duke@435 291

mercurial