[cfe-dev] [llvm-dev] PATCH: clang/LLVM - Sparc - inline ASM with floating pointer registers
James Y Knight via cfe-dev
cfe-dev at lists.llvm.org
Tue Mar 22 08:27:38 PDT 2016
Your change looks almost correct.
- Test cases.
- Add the new 'f' type to the documentation in LangRef.rst.
- Check the type of the arg (f32, f64, or f128) to choose the right one of
FPRegsRegClass, DFPRegsRegClass, QFPRegsRegClass.
Then, please submit for review, per
(Typically best to do so via Phabricator,
On Mon, Mar 21, 2016 at 6:46 PM, Arnaud Allard de Grandmaison via llvm-dev <
llvm-dev at lists.llvm.org> wrote:
> Hi Patrick,
> I do not see a problem with those patches --- apart from missing tests ;)
> On Mon, Mar 21, 2016 at 3:11 PM, Patrick Boettcher via cfe-dev <
> cfe-dev at lists.llvm.org> wrote:
>> Hi lists,
>> Sorry for cross-posting, but these two patches are of concern to clang
>> and llvm.
>> I'm compiling C-code which includes inline assembler which looks like
>> the following using the sparc-target:
>> static inline float fabsf(float a)
>> float res;
>> __asm __volatile__("fabss %1, %0;"
>> : "=f"(res)
>> : "f"(a));
>> return res;
>> This fails with llvm/clang - trunk/master.
>> I patched clang and llvm and, as I'm new to both project, I'm having
>> doubts that it is enough - however it works.
>> What would be necessary to have it included in both projects?
>> Is my code correct for what I want to do?
>> What would have been a more appropriate process to submit?
>> best regards,
>> cfe-dev mailing list
>> cfe-dev at lists.llvm.org
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-dev