[Webkit-unassigned] [Bug 162705] Offline asm should not output masm assembly when using a x86_64 asm backend

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=162705

--- Comment #12 from Mark Lam <mark.lam at apple.com> ---
Comment on attachment 290146
  --> https://bugs.webkit.org/attachment.cgi?id=290146
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=290146&action=review

>> 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.

Also remove all the other unnecessary changes if they are indeed unnecessary.  Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160929/44135e3a/attachment.html>