[clang] [FMV] Emit the resolver along with the default version definition. (PR #84405)

Jon Roelofs via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 14 15:45:31 PDT 2024


================
@@ -4120,6 +4127,9 @@ void CodeGenModule::emitMultiVersionFunctions() {
                                    TA->getArchitecture(), Feats);
             } else {
               const auto *TVA = CurFD->getAttr<TargetVersionAttr>();
+              if (TVA->isDefaultVersion() &&
+                  CurFD->doesThisDeclarationHaveABody())
+                EmitResolver = true;
               llvm::SmallVector<StringRef, 8> Feats;
               TVA->getFeatures(Feats);
               Options.emplace_back(cast<llvm::Function>(Func),
----------------
jroelofs wrote:

github won't let me put a comment on the line I want to, but should there be a `EmitResolver = true` down in `} else if (FD->isTargetClonesMultiVersion()) {` ?

https://github.com/llvm/llvm-project/pull/84405


More information about the cfe-commits mailing list