[PATCH] D42962: [ARM] Allow 64- and 128-bit types with 't' inline asm constraint

Pablo Barrio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 10:45:43 PST 2018


pbarrio added a comment.

> AFAICS, the current approach just checks the size of the type, not the size of the sub-type. f64 or even integer types could still leak in, no?
> 
> To prove they're not, we need tests making sure they break if you try.

Ah, yes, totally right in that, good call. I'll add more testing.


Repository:
  rL LLVM

https://reviews.llvm.org/D42962





More information about the llvm-commits mailing list