[llvm] r288592 - [PM] Make PreservedAnalyses::preserved take its parameter by const ref.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 3 11:49:12 PST 2016


Author: jlebar
Date: Sat Dec  3 13:49:12 2016
New Revision: 288592

URL: http://llvm.org/viewvc/llvm-project?rev=288592&view=rev
Log:
[PM] Make PreservedAnalyses::preserved take its parameter by const ref.

Summary: Previously we were unnecessarily copying the argument.

Reviewers: chandlerc

Subscribers: mehdi_amini, silvas, llvm-commits

Differential Revision: https://reviews.llvm.org/D27368

Modified:
    llvm/trunk/include/llvm/IR/PassManager.h

Modified: llvm/trunk/include/llvm/IR/PassManager.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/PassManager.h?rev=288592&r1=288591&r2=288592&view=diff
==============================================================================
--- llvm/trunk/include/llvm/IR/PassManager.h (original)
+++ llvm/trunk/include/llvm/IR/PassManager.h Sat Dec  3 13:49:12 2016
@@ -137,7 +137,7 @@ public:
   }
 
   /// \brief Query whether all of the analyses in the set are preserved.
-  bool preserved(PreservedAnalyses Arg) {
+  bool preserved(const PreservedAnalyses& Arg) {
     if (Arg.areAllPreserved())
       return areAllPreserved();
     for (auto ID : Arg.PreservedAnalysisIDs)




More information about the llvm-commits mailing list