[PATCH] D43125: [ARM] Don't print "Requires NEON" error message for M-profile

Renato Golin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 12 03:19:54 PST 2018


rengolin added a comment.

For M-class, "invalid instruction" is indeed more accurate than requires NEON. We don't want people to think they can add "-mfpu=neon" or ".fpu neon" to an M-class to force assembling that code.

To have a better handling than this, we'd have to error out earlier saying M-class doesn't have NEON, which probably needs more knowledge than that piece of code has... LGTM too, for now.


https://reviews.llvm.org/D43125





More information about the llvm-commits mailing list