<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div><br></div><div>You should be able to do something like:</div><div><br></div><div># Turn off aliasing with GCC for ExtensionSubtables.cpp</div><div>ifeq ($(PLATFORM), linux)</div><div> CXXFLAGS += $(CXXFLAGS_$(@F))<br></div><div><div> CC_NEWER_THAN_43 := \</div><div> $(shell $(EXPR) $(CC_MAJORVER) \> 4 \| \</div><div> \( $(CC_MAJORVER) = 4 \& $(CC_MINORVER) \> 3 \) )</div><div> ifeq ($(CC_NEWER_THAN_43),1)</div> CXXFLAGS_ExtensionSubtables.o = -fno-strict-aliasing</div><div> endif<br>endif</div></div><div><br></div><div>-kto</div><div><br></div><div><div>On May 29, 2012, at 6:50 AM, Phil Race wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<div bgcolor="#FFFFFF" text="#000000">
I have one concern<br>
<br>
It sounds like the option is new in gcc 4.4 :
<a class="moz-txt-link-freetext" href="http://gcc.gnu.org/gcc-4.4/porting_to.html">http://gcc.gnu.org/gcc-4.4/porting_to.html</a><br>
<br>
I am not sure you can assume that since ..<br>
<br>
<a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/jdk8/jdk8/raw-file/tip/README-builds.html#gcc">http://hg.openjdk.java.net/jdk8/jdk8/raw-file/tip/README-builds.html#gcc</a><br>
"
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
The GNU gcc compiler version should be 4.3 or newer."<br>
<br>
And I suggest a push to 2d-dev since its in the 2d code we get<br>
a warning and the "build" change is in 2D build files, specifically
to work<br>
around that warning .. its really precious little to do with the
build system.<br>
<br>
-phil.<br>
<br>
On 5/29/12 6:13 AM, Andrew Hughes wrote:
<blockquote cite="mid:6c755de2-ebf0-4cb0-a231-1ca0e81e442f@zmail17.collab.prod.int.phx2.redhat.com" type="cite">
<pre wrap="">----- Original Message -----
</pre>
<blockquote type="cite">
<pre wrap="">In anticipation of the updated fix I created
7171223 : Building ExtensionSubtables.cpp should use
-fno-strict-aliasing
-phil.
</pre>
<blockquote type="cite">
<pre wrap="">Ok for jdk8? If so, can I have a bug ID for this please?
Thanks,
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<pre wrap="">Thanks for the bug ID, Phil.
Here's the revised webrev:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~andrew/no-strict-aliasing/webrev.02/">http://cr.openjdk.java.net/~andrew/no-strict-aliasing/webrev.02/</a>
Sorry for the delay. I found when trying to build the build tree
that HotSpot had regressed and become unbuildable:
<a class="moz-txt-link-freetext" href="http://mail.openjdk.java.net/pipermail/hotspot-dev/2012-May/005945.html">http://mail.openjdk.java.net/pipermail/hotspot-dev/2012-May/005945.html</a>
All patched now. Is build the preferred tree to push this too,
or would you prefer somewhere else?
Thanks,
</pre>
</blockquote>
<br>
</div>
</blockquote></div><br></body></html>