<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Crash on xLarge memory allocation using bmalloc on 32bit systems"
href="https://bugs.webkit.org/show_bug.cgi?id=146440#c16">Comment # 16</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Crash on xLarge memory allocation using bmalloc on 32bit systems"
href="https://bugs.webkit.org/show_bug.cgi?id=146440">bug 146440</a>
from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=256038&action=diff" name="attach_256038" title="Patch proposal">attachment 256038</a> <a href="attachment.cgi?id=256038&action=edit" title="Patch proposal">[details]</a></span>
Patch proposal
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=256038&action=review">https://bugs.webkit.org/attachment.cgi?id=256038&action=review</a>
Do you know how important this optimization is? My main concern, like with all other patches that disable optimizations to work around compiler bugs, is that it's very difficult to know when we can stop disabling it, and we end up with the optimization disabled forever. Also, I think we should report these bugs to GCC as well.
<span class="quote">>>> Source/WebCore/CMakeLists.txt:3581
>>> +if (CMAKE_COMPILER_IS_GNUCXX AND "${LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR}" MATCHES "(i[3-6]86|x86)")
>>
>> I think this should be: ... MATCHES "(i[3-6]86|x86)$")
>>
>> Otherwise this becomes true also for AMD64 with GCC (where LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR = x86_64 ). (yes, there is a bug on the previous code introduced on 128855 <<a href="http://trac.webkit.org/r128855">http://trac.webkit.org/r128855</a>>)
>
> Also I don't think this will work for ARM 32-bit machines or $RANDOMARCH 32-bit machines.</span >
Yes, we do a similar match to add the --no-keep-memory linker option in 32 bits. I think we could move it to a common place, detecting also non intel 32 bits processors and exposing a variable to CMake.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>