[PATCH] D146059: [SystemZ] Allow fp/int casting into inline assembly operands

[Ilya Leoshkevich via Phabricator via llvm-commits]

iii added a comment.

I can't really comment on the implementation, but the examples in the testcase looks good to me.



================
Comment at: llvm/test/CodeGen/SystemZ/inline-asm-fp-int-casting.ll:43
+  %cc_dep1 = load i128, ptr %0, align 8
+  %1 = tail call i128 asm sideeffect "", "=f,0"(i128 %cc_dep1)
+  store i128 %1, ptr %agg.result, align 8
----------------
iii wrote:
> Did this work for you in GCC? I tried:
> 
>     __int128 foo(__int128 bar) {
>         asm("" : "+f" (bar));
>         return bar;
>     }
> 
> and got:
> 
>     error: inconsistent operand constraints in an ‘asm’
> 
> I wanted to give it a try, because I would expect `ld`/`std` here, like for long double.
My bad, I should have specified `-march=z13`.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146059/new/

https://reviews.llvm.org/D146059