[PATCH] D53748: [ARM] Fix test inlineasm-X-allocation.ll

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 26 04:18:26 PDT 2018


SjoerdMeijer added a comment.

> However, it seems wrong for the compiler to assign s0 as an operand without having FP registers available.

Okay, to be more precise, I don't understand what exactly we are expecting here. Why are we expecting `vadd.f32` to work on integer registers? What I mean is that I see `-neon` is specified, but why in the absense of that do we expect integer registers here. That doesn't make much sense for `vadd.f32`, isn't it?


https://reviews.llvm.org/D53748





More information about the llvm-commits mailing list