[PATCH] D32281: [ARM] ACLE Chapter 9 intrinsics

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 12:52:38 PDT 2017


efriedma added a comment.

Either we're trying to support reading the Q flag, or we're not trying to support it.  If we are trying to support it, we should do it correctly; if we aren't trying to support it, we might as well mark the intrinsics which write to it as IntrNoMem.  There isn't really a middle ground here.

Also, I'd like to see comments on each of the intrinsics which isn't IntrNoMem describing which bits it reads/writes.


https://reviews.llvm.org/D32281





More information about the llvm-commits mailing list