================
@@ -2301,8 +2301,8 @@ void ModuleMapLoader::handleUseDecl(const 
modulemap::UseDecl &UD) {
 }
 
 void ModuleMapLoader::handleLinkDecl(const modulemap::LinkDecl &LD) {
-  ActiveModule->LinkLibraries.push_back(
-      Module::LinkLibrary(std::string{LD.Library}, LD.Framework));
+  ActiveModule->LinkLibraries.emplace_back(std::string{LD.Library},
+                                           LD.Framework);
----------------
hardikxk wrote:

Not really simply missed it before as I got diverted with the previous 
constructor changes. Thanks for the review!! I've replaced it in the latest 
commit.

https://github.com/llvm/llvm-project/pull/232260
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to