<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Offline asm should not output masm assembly when using a x86_64 asm backend"
href="https://bugs.webkit.org/show_bug.cgi?id=162705#c12">Comment # 12</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Offline asm should not output masm assembly when using a x86_64 asm backend"
href="https://bugs.webkit.org/show_bug.cgi?id=162705">bug 162705</a>
from <span class="vcard"><a class="email" href="mailto:mark.lam@apple.com" title="Mark Lam <mark.lam@apple.com>"> <span class="fn">Mark Lam</span></a>
</span></b>
<pre>Comment on <span class=""><a href="attachment.cgi?id=290146&action=diff" name="attach_290146" title="Patch">attachment 290146</a> <a href="attachment.cgi?id=290146&action=edit" title="Patch">[details]</a></span>
Patch
View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=290146&action=review">https://bugs.webkit.org/attachment.cgi?id=290146&action=review</a>
<span class="quote">>> Source/JavaScriptCore/offlineasm/x86.rb:142
>> end
>
> It looks like this change is the only one that is of consequence. All the other changes below only change the code to use a local var (initialized by a function) instead of a global var. Are they really necessary? Or did I misread something?
>
> I see that the isGCC and isMSVC functions are also based on isCompilingOnWindows. isGCC is not used anywhere. isMSVC is used for (1) determining the offlineasm output file format and (2) whether to enable DWARF2 debug annotations. If you're cross compiling with Clang for a non-Windows target, wouldn't you want these configured like other clang builds too?
>
> This is what I would do:
> 1. remove isGCC because it is not used.
> 2. change isMSVC to be based on isWin.
> 3. remove isCompilingOnWindows since it will no longer be in use.</span >
Also remove all the other unnecessary changes if they are indeed unnecessary. Thanks.</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>