<div dir="ltr">Fair enough, I think my objection was mostly on the grounds that I'm not familiar with the extractSymbols script, but if this patch doesn't do anything that isn't already pervasive elsewhere, then I can't really object.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 13, 2016 at 9:00 AM Mehdi AMINI via Phabricator <<a href="mailto:reviews@reviews.llvm.org">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">mehdi_amini added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I don't have anything against this patch as is, but I don't know about the MSVC issue so I won't be able to help here.<br class="gmail_msg">
<br class="gmail_msg">
In <a href="https://reviews.llvm.org/D26671#613437" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26671#613437</a>, @zturner wrote:<br class="gmail_msg">
<br class="gmail_msg">
> I see.  My mild preference would be to remove as many C++isms as possible from exported interfaces.  So, in descending order of preference, this would be:<br class="gmail_msg">
><br class="gmail_msg">
> 1. Make it return a `void*` instead of a `fltSemantics&`<br class="gmail_msg">
> 2. Make these global free functions marked extern "C"<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
If you want a C-like API, you should extend the LLVM C API or write wrappers around the C++ API, I don't believe there is a wide agreement to "remove C++isms" from LLVM C++ APIs themselves (it is possible I misunderstood what you're suggesting here).<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: llvm/include/llvm/ADT/APFloat.h:143<br class="gmail_msg">
+  static const fltSemantics &PPCDoubleDouble();<br class="gmail_msg">
+  static const fltSemantics &x87DoubleExtended();<br class="gmail_msg">
<br class="gmail_msg">
----------------<br class="gmail_msg">
Functions name in LLVM usually starts with a verb (getters start with  `get`).<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D26671" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D26671</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>