<div dir="ltr">Your change looks almost correct.<div><br></div><div>You need:</div><div>- Test cases.</div><div>- Add the new 'f' type to the documentation in LangRef.rst.</div><div>- Check the type of the arg (f32, f64, or f128) to choose the right one of FPRegsRegClass, DFPRegsRegClass, QFPRegsRegClass.</div><div><br></div><div>Then, please submit for review, per <a href="http://llvm.org/docs/DeveloperPolicy.html">http://llvm.org/docs/DeveloperPolicy.html</a></div><div>(Typically best to do so via Phabricator, <a href="http://llvm.org/docs/Phabricator.html">http://llvm.org/docs/Phabricator.html</a>)</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 21, 2016 at 6:46 PM, Arnaud Allard de Grandmaison via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</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">Hi Patrick,<div><br></div><div>I do not see a problem with those patches --- apart from missing tests ;)</div><div><br></div><div>Cheers,</div><div>Arnaud</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 21, 2016 at 3:11 PM, Patrick Boettcher via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi lists,<br>
<br>
Sorry for cross-posting, but these two patches are of concern to clang<br>
and llvm.<br>
<br>
I'm compiling C-code which includes inline assembler which looks like<br>
the following using the sparc-target:<br>
<br>
static inline float fabsf(float a)<br>
{<br>
  float res;<br>
  __asm __volatile__("fabss  %1, %0;"<br>
                     : "=f"(res)<br>
                     : "f"(a));<br>
  return res;<br>
}<br>
<br>
This fails with llvm/clang - trunk/master.<br>
<br>
I patched clang and llvm and, as I'm new to both project, I'm having<br>
doubts that it is enough - however it works.<br>
<br>
What would be necessary to have it included in both projects?<br>
Is my code correct for what I want to do?<br>
What would have been a more appropriate process to submit?<br>
<br>
best regards,<br>
--<br>
Patrick.<br>
<br>
<br>
<br>
<br>
<br>
<br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>