Author: erichkeane Date: 2025-04-25T09:45:06-07:00 New Revision: 785ab45c2b300ba6291d8fb21762eabc050275f6
URL: https://github.com/llvm/llvm-project/commit/785ab45c2b300ba6291d8fb21762eabc050275f6 DIFF: https://github.com/llvm/llvm-project/commit/785ab45c2b300ba6291d8fb21762eabc050275f6.diff LOG: [NFC] move comment back to where it belongs. It appears that OpenMP changes caused this comment to get moved away from where it was intended, so this patch puts it back next to the multiversioning code, as intended. Added: Modified: clang/lib/CodeGen/CodeGenModule.cpp Removed: ################################################################################ diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp index b46eaf3755438..e917f3c42da06 100644 --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -4691,8 +4691,6 @@ llvm::Constant *CodeGenModule::GetOrCreateLLVMFunction( const Decl *D = GD.getDecl(); std::string NameWithoutMultiVersionMangling; - // Any attempts to use a MultiVersion function should result in retrieving - // the iFunc instead. Name Mangling will handle the rest of the changes. if (const FunctionDecl *FD = cast_or_null<FunctionDecl>(D)) { // For the device mark the function as one that should be emitted. if (getLangOpts().OpenMPIsTargetDevice && OpenMPRuntime && @@ -4710,6 +4708,8 @@ llvm::Constant *CodeGenModule::GetOrCreateLLVMFunction( } } + // Any attempts to use a MultiVersion function should result in retrieving + // the iFunc instead. Name Mangling will handle the rest of the changes. if (FD->isMultiVersion()) { UpdateMultiVersionNames(GD, FD, MangledName); if (!IsForDefinition) { _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
