<div dir="ltr">@zturner, yes, it is not well documented. Because of that I propose yo consider `dumpbin` first, and `link <span style="font-size:12.8px"> /dump /disasm</span>` as second option in case we don't find `dumpbin`.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 17, 2017 at 10:00 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">By `link`, do you mean the MSVC  linker?  I didn't know it supported /dump and /disasm options, and when I run link /help I don't see those options listed.</div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 13, 2017 at 1:53 PM Marcos Pividori via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mpividori added inline comments.<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
================<br class="m_7883762344524539890gmail_msg">
Comment at: lib/Fuzzer/FuzzerTracePC.cpp:<wbr>162<br class="m_7883762344524539890gmail_msg">
+      size_t PcOffsetEnd = S.find(':');<br class="m_7883762344524539890gmail_msg">
+      if (PcOffsetEnd == std::string::npos) continue;<br class="m_7883762344524539890gmail_msg">
+      S.resize(PcOffsetEnd);<br class="m_7883762344524539890gmail_msg">
----------------<br class="m_7883762344524539890gmail_msg">
zturner wrote:<br class="m_7883762344524539890gmail_msg">
> Can you put the `continue` on a new line?<br class="m_7883762344524539890gmail_msg">
Ok, I will do that. But, in line 169, the continue is in the same line too. Do you want me to update that too?<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
================<br class="m_7883762344524539890gmail_msg">
Comment at: lib/Fuzzer/FuzzerUtilWindows.<wbr>cpp:181<br class="m_7883762344524539890gmail_msg">
+std::string DisassembleCmd(const std::string &FileName) {<br class="m_7883762344524539890gmail_msg">
+  return "dumpbin /disasm " + FileName;<br class="m_7883762344524539890gmail_msg">
+}<br class="m_7883762344524539890gmail_msg">
----------------<br class="m_7883762344524539890gmail_msg">
zturner wrote:<br class="m_7883762344524539890gmail_msg">
> `dumpbin` is not guaranteed to be in the user's path.  Is this going to cause a problem?  Also I believe `dumpbin` will use Intel syntax for the disassembly.  Does objdump use AT&T syntax by default?  If so, will this cause a problem?<br class="m_7883762344524539890gmail_msg">
@zturner you are right. I didn't realize of that since I was using a terminal with the VS environment.<br class="m_7883762344524539890gmail_msg">
We can use :  `link /dump /disasm` , which is included in PATH, but I couldn't find information if the `/dump` option is a new option or it is also included in old versions.<br class="m_7883762344524539890gmail_msg">
Also, I think we should consider the case when users only using llvm tools, so in that case we could look for: `llvm-objdump`<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
So, I propose to do this:<br class="m_7883762344524539890gmail_msg">
First: See if `dumpbin` is present in PATH,<br class="m_7883762344524539890gmail_msg">
Else: See if `link`  is present in PATH,<br class="m_7883762344524539890gmail_msg">
Else: See if `llvm-objdump` is present in PATH,<br class="m_7883762344524539890gmail_msg">
Else: Fail<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
Would you agree?<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
Repository:<br class="m_7883762344524539890gmail_msg">
  rL LLVM<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
<a href="https://reviews.llvm.org/D28635" rel="noreferrer" class="m_7883762344524539890gmail_msg" target="_blank">https://reviews.llvm.org/<wbr>D28635</a><br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
<br class="m_7883762344524539890gmail_msg">
</blockquote></div>
</div></div></blockquote></div><br></div>