[llvm] Convert copy to r-value (PR #89053)

Chris B via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 17 16:21:28 PDT 2024


================
@@ -168,7 +168,7 @@ void PSVRuntimeInfo::finalize(Triple::EnvironmentType Stage) {
   DXConStrTabBuilder.add(EntryName);
 
   DXConStrTabBuilder.finalize();
-  for (auto ElAndName : zip(SignatureElements, SemanticNames)) {
+  for (auto &&ElAndName : zip(SignatureElements, SemanticNames)) {
----------------
llvm-beanz wrote:

The copy here would just be copying a `std::tuple` of iterators (which are pointers). That doesn’t seem like something that should cause any performance issues, and this doesn’t match other idiomatic use cases of `llvm::zip`.

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


More information about the llvm-commits mailing list