[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 05:17:19 PST 2018


rengolin added a comment.

In https://reviews.llvm.org/D43125#1004879, @olista01 wrote:

> We actually could do that here, the code immediately following the change is building the string to be printed, so we could just return a special "M-class doesn't have NEON" string. However, that would probably be a special-case too far, so I think this is fine as it is.


Yeah, this would have to be a table-gen driven kind of change to cover all cases. For another patch.


https://reviews.llvm.org/D43125





More information about the llvm-commits mailing list