[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 #17 from Christopher Reid <Christopher.reid at sony.com> ---
Created attachment 290282
  --> https://bugs.webkit.org/attachment.cgi?id=290282&action=review
Work in progress patch

Here's a WIP patch for supporting isMSVC to use isWIN on the asm.rb side. I moved the comment style to be specific to which backend code is being generated for. 

> +    $output.puts("//" + inputHash)
The one problem I had was it's a chicken and an egg problem for which comment style to use for the initial hash comment. Currently, we need to use either ';' or '//' to store the hashes to check to see if we need to re-build. We don't know what backends we're building for until we start parsing the file and we need to put the hash comment before we start generating outputs for each backend. There will also be problems if there's a mix of backends using different comment styles.

Here's some possible solutions for this problem:
- Buffer the output and write afer we've generated code for all backends and customize the comment then
- Switch to a new way to check for changes rather than storing hashes in comments
- Output the comment based on the first backend we use

Do you have any other ideas or input on how I should go about this?

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/20160930/0c17f5ac/attachment.html>