[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