<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jan 31, 2018 at 7:23 PM, Paul Robinson via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">probinson added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D42758#993936" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D42758#993936</a>, @ruiu wrote:<br>
<br>
> > I also wonder which is better `#pragma comment(lib, "m")` or `#pragma comment(lib, "m")`.<br>
><br>
> Sorry, I meant `#pragma comment(lib, "m")` or `#pragma comment("lib", "m")`.<br>
<br>
<br>
</span>I can't swear to it but I don't think Microsoft invented `#pragma comment`.  Various IBM compilers have it, with a syntax of `#pragma comment ( <keyword> [ , "string" ] )`.  I'm not seeing a `lib` keyword specifically in the IBM docs, but being a keyword would be consistent with past practice.<br></blockquote><div><br></div><div>Thank you for the info. If it's a common practice, I don't see a reason to change that. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
================<br>
Comment at: lib/Parse/ParsePragma.cpp:299<br>
<span class="">+      getTargetInfo().getTriple().<wbr>isOSBinFormatELF()) {<br>
     MSCommentHandler.reset(new PragmaCommentHandler(Actions))<wbr>;<br>
     PP.AddPragmaHandler(<wbr>MSCommentHandler.get());<br>
</span>----------------<br>
PS4's binary format is ELF, so you should be able to remove the isPS4 predicate.<br>
<br>
<br>
================<br>
Comment at: lib/Parse/ParsePragma.cpp:382<br>
+      getTargetInfo().getTriple().<wbr>isOSBinFormatELF()) {<br>
     PP.RemovePragmaHandler(<wbr>MSCommentHandler.get());<br>
     MSCommentHandler.reset();<br>
----------------<br>
Don't need isPS4, as it uses ELF.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rC Clang<br>
<br>
<a href="https://reviews.llvm.org/D42758" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D42758</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>