---

yaml
---
r: 60795
b: refs/heads/auto
c: 998e2ce
h: refs/heads/master
i:
  60793: e1bde2c
  60791: 6789568
v: v3

Author: Aatch

[refs]

@@ -14,6 +14,6 @@ refs/heads/try3: 9387340aab40a73e8424c48fd42f0c521a4875c0
 refs/tags/release-0.3.1: 495bae036dfe5ec6ceafd3312b4dca48741e845b
 refs/tags/release-0.4: e828ea2080499553b97dfe33b3f4d472b4562ad7
 refs/tags/release-0.5: 7e3bcfbf21278251ee936ad53e92e9b719702d73
-refs/heads/auto: 4988c89e86d7120d5c9b0d5ccabba4e8763e1fe7
+refs/heads/auto: 998e2ce4ff7ae7ab1227a30ed3e20e503da53b27
 refs/heads/servo: af82457af293e2a842ba6b7759b70288da276167
 refs/tags/release-0.6: b4ebcfa1812664df5e142f0134a5faea3918544c

branches/auto/src/librustc/back/passes.rs

@@ -55,12 +55,13 @@ impl PassManager {
 
 pub fn populatePassManager(pm: &mut PassManager, level:session::OptLevel) {
     unsafe {
+        // We add a lot of normally-unused prototypes, so always strip them
+        // straight away, later passes will get rid of any that are optimized
+        // away
+        pm.addPass(llvm::LLVMCreateStripDeadPrototypesPass());
         if level == session::No {
             pm.addPass(llvm::LLVMCreateAlwaysInlinerPass());
 
-            // We add a lot of unused prototypes, so strip them no matter
-            // what
-            pm.addPass(llvm::LLVMCreateStripDeadPrototypesPass());
             return;
         }
 
@@ -81,6 +82,13 @@ pub fn populatePassManager(pm: &mut PassManager, level:session::OptLevel) {
 
         pm.addPass(llvm::LLVMCreatePruneEHPass());
 
+        if level == session::Aggressive {
+            // Do this before inlining, since inlining might
+            // make minor changes to functions that mean they
+            // can't be merged, despite being almost identical
+            pm.addPass(llvm::LLVMCreateMergeFunctionsPass());
+        }
+
         match level {
             session::Less       => pm.addPass(llvm::LLVMCreateFunctionInliningPass(200)),
             session::Default    => pm.addPass(llvm::LLVMCreateFunctionInliningPass(225)),
@@ -94,8 +102,6 @@ pub fn populatePassManager(pm: &mut PassManager, level:session::OptLevel) {
             pm.addPass(llvm::LLVMCreateArgumentPromotionPass());
         }
 
-        pm.addPass(llvm::LLVMCreateSROAPass());
-
         pm.addPass(llvm::LLVMCreateEarlyCSEPass());
         pm.addPass(llvm::LLVMCreateSimplifyLibCallsPass());
         pm.addPass(llvm::LLVMCreateJumpThreadingPass());
@@ -109,8 +115,6 @@ pub fn populatePassManager(pm: &mut PassManager, level:session::OptLevel) {
         pm.addPass(llvm::LLVMCreateLoopRotatePass());
         pm.addPass(llvm::LLVMCreateLICMPass());
 
-        pm.addPass(llvm::LLVMCreateLoopUnswitchPass());
-
         pm.addPass(llvm::LLVMCreateInstructionCombiningPass());
         pm.addPass(llvm::LLVMCreateIndVarSimplifyPass());
         pm.addPass(llvm::LLVMCreateLoopIdiomPass());
@@ -134,16 +138,17 @@ pub fn populatePassManager(pm: &mut PassManager, level:session::OptLevel) {
 
         pm.addPass(llvm::LLVMCreateBBVectorizePass());
         pm.addPass(llvm::LLVMCreateInstructionCombiningPass());
+        pm.addPass(llvm::LLVMCreateEarlyCSEPass());
+
+        pm.addPass(llvm::LLVMCreateLoopUnrollPass());
+
+        pm.addPass(llvm::LLVMCreateAggressiveDCEPass());
+        pm.addPass(llvm::LLVMCreateCFGSimplificationPass());
+        pm.addPass(llvm::LLVMCreateInstructionSimplifierPass());
+
         if level != session::Less {
             pm.addPass(llvm::LLVMCreateGlobalDCEPass());
             pm.addPass(llvm::LLVMCreateConstantMergePass());
         }
-
-        if level == session::Aggressive {
-            pm.addPass(llvm::LLVMCreateMergeFunctionsPass());
-        }
-
-        pm.addPass(llvm::LLVMCreateStripDeadPrototypesPass());
-
     }
 }