<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body><span class="vcard"><a class="email" href="mailto:Christopher.reid@sony.com" title="Christopher Reid <Christopher.reid@sony.com>"> <span class="fn">Christopher Reid</span></a>
</span> changed
<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>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #290250 is obsolete</td>
<td>
</td>
<td>1
</td>
</tr>
<tr>
<td style="text-align:right;">Attachment #290282 is obsolete</td>
<td>
</td>
<td>1
</td>
</tr>
<tr>
<td style="text-align:right;">Attachment #290250 Flags</td>
<td>review?
</td>
<td>
</td>
</tr>
<tr>
<td style="text-align:right;">Attachment #290396 Flags</td>
<td>
</td>
<td>review?
</td>
</tr></table>
<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#c25">Comment # 25</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:Christopher.reid@sony.com" title="Christopher Reid <Christopher.reid@sony.com>"> <span class="fn">Christopher Reid</span></a>
</span></b>
<pre>Created <span class=""><a href="attachment.cgi?id=290396&action=diff" name="attach_290396" title="Patch to remove the hash">attachment 290396</a> <a href="attachment.cgi?id=290396&action=edit" title="Patch to remove the hash">[details]</a></span>
Patch to remove the hash
Here's a patch I made to remove the hash comment for reference.
It should work okay with cmake as cmake rebuilds based on changes in dependencies.
I'm not working on mac so I don't know how xcode handles re-generation but it might always regenerate the parse tree with this change.
<span class="quote">>Why not just add an optional argument you could use when building that would say "override the assembler to be xxx"?</span >
When I first started looking at this task, I looked in to making this change as a command line option. Using the build rather than the ENV OS check to automatically determine the type of asm felt a lot cleaner to me.
There are a few changes I might have to make when dealing with arguments and I was wondering which approach I should take for them.
Right now the script expects input/output files in a specific order as the first 3 arguments.
I think I would have to either remake those arguments in to --FILE=X args or the new argument would need to always be after those arguments.
The input hash would have to take in to account the new possibility that the output asm can change based on an argument.
To solve this, I would have to hash the arguments passed. If I make the change that arguments can be in any order, it's a small issue but I think arguments would need to be sorted before hashing since the order of arguments doesn't matter.
Should I continue along the lines of this patch or should I work towards making this a command line change?
I'm not sure which would be the best approach if wincairo is switching to clang.
Does visual studio's clang use the masm assembler? I agree a command line option might be best if that's the case.</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>