Yeah given the evidence, maybe it's fine left as is. I worked around it locally so I am not blocked. Thanks!<br><br>On Monday, March 11, 2019, Jonas Devlieghere <<a href="mailto:jonas@devlieghere.com">jonas@devlieghere.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">It's weird that this only occurs when building with Xcode, none of the bots seem affected. Anyway, I'm updating the macros and verifying the build works locally when using Xcode.   </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 10, 2019 at 3:25 PM Dave Lee 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">kastiglione added a comment.<br>
<br>
I'm getting build failures that appear to be caused by this patch. I'm compiling on macOS with Xcode 10.1. The failures are uses of `LLDB_RECORD_DUMMY()`, passing a callback function pointer as one of the args. This results in errors like:<br>
<br>
  ../../tools/lldb/include/lldb/<wbr>Utility/<wbr>ReproducerInstrumentation.h:<wbr>37:6: error: invalid operands to binary expression ('llvm::raw_string_ostream' and 'bool (*)(void *, lldb::SBProcess &, lldb::SBThread &, lldb::SBBreakpointLocation &)')<br>
    ss << t;<br>
    ~~ ^  ~<br>
<br>
and<br>
<br>
  ../../include/llvm/Support/<wbr>raw_ostream.h:202:16: note: candidate function not viable: no known conversion from 'bool (*)(void *, lldb::SBProcess &, lldb::SBThread &, lldb::SBBreakpointLocation &)' to 'const void *' for 1st argument; take the address of the argument with &<br>
    raw_ostream &operator<<(const void *P);<br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D59101/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D59101/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D59101" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D59101</a><br>
<br>
<br>
<br>
</blockquote></div>
</blockquote>