[NFC] Fix 'target' condition in checkTargetFeatures
authorErich Keane <erich.keane@intel.com>
Mon, 18 Nov 2019 21:38:56 +0000 (13:38 -0800)
committerErich Keane <erich.keane@intel.com>
Mon, 18 Nov 2019 21:43:52 +0000 (13:43 -0800)
commit0213adde218530bc31e5c4e50b49704c6bb2f2e9
treec88004261b31d31e33bcc1a1d7b1de73175fe830
parent085486779890e394dd42447c7c5b0fb36a79ea2b
[NFC] Fix 'target' condition in checkTargetFeatures

checkTargetFeatures was incorrectly checking for cpu_specific instead of
just 'target'. While this function was never called in that situation,
it seemed correct to fix the condition.  Additionally, multiversion
functions can never be always_inline, but if any function accidentially
ended up here we shouldn't diagnose.

Note that the adding of target-features to the list is unnecessary since
the getFunctionFeatureMap actually considers attribute target,
however adding it results in significantly better error messages by
putting the 'target' features first (and thus first to fail).
Otherwise, the error message would be the first feature 'implied' by the
target attribute, and not necessarily the feature listed in the
attribute itself.
clang/lib/CodeGen/CodeGenFunction.cpp