Changeset View
Changeset View
Standalone View
Standalone View
lib/CodeGen/TargetPassConfig.cpp
| Show All 40 Lines | |||||
| 41 | #include "llvm/Support/Threading.h" | 41 | #include "llvm/Support/Threading.h" | ||
| 42 | #include "llvm/Target/TargetMachine.h" | 42 | #include "llvm/Target/TargetMachine.h" | ||
| 43 | #include "llvm/Transforms/Scalar.h" | 43 | #include "llvm/Transforms/Scalar.h" | ||
| 44 | #include "llvm/Transforms/Utils/SymbolRewriter.h" | 44 | #include "llvm/Transforms/Utils/SymbolRewriter.h" | ||
| 45 | #include <cassert> | 45 | #include <cassert> | ||
| 46 | #include <string> | 46 | #include <string> | ||
| 47 | 47 | | |||
| 48 | using namespace llvm; | 48 | using namespace llvm; | ||
| 49 | 49 | | |||
| 50 | static cl::opt<bool> DisablePostRASched("disable-post-ra", cl::Hidden, | 50 | static cl::opt<bool> DisablePostRASched("disable-post-ra", cl::Hidden, | ||
iteratee: These seem unrelated. Can you pull them out? | |||||
Not Done ReplyI do not understand why you want me to pull these out. They were not added by me and it is being used later on in this src. lei: I do not understand why you want me to pull these out. They were not added by me and it is… | |||||
Not Done ReplySorry, I was trying to look at your original diff vs this one to see what changed. iteratee: Sorry, I was trying to look at your original diff vs this one to see what changed. | |||||
Not Done ReplyWhat are you talking about? I see no changes here. hfinkel: What are you talking about? I see no changes here. | |||||
Not Done ReplyIf you compare diff 4 to diff 5 it makes it look like EnableIPRA got added in diff 5. sfertile: If you compare diff 4 to diff 5 it makes it look like EnableIPRA got added in diff 5. | |||||
| 51 | cl::desc("Disable Post Regalloc Scheduler")); | 51 | cl::desc("Disable Post Regalloc Scheduler")); | ||
| 52 | static cl::opt<bool> DisableBranchFold("disable-branch-fold", cl::Hidden, | 52 | static cl::opt<bool> DisableBranchFold("disable-branch-fold", cl::Hidden, | ||
| 53 | cl::desc("Disable branch folding")); | 53 | cl::desc("Disable branch folding")); | ||
| 54 | static cl::opt<bool> DisableTailDuplicate("disable-tail-duplicate", cl::Hidden, | 54 | static cl::opt<bool> DisableTailDuplicate("disable-tail-duplicate", cl::Hidden, | ||
| 55 | cl::desc("Disable tail duplication")); | 55 | cl::desc("Disable tail duplication")); | ||
| 56 | static cl::opt<bool> DisableEarlyTailDup("disable-early-taildup", cl::Hidden, | 56 | static cl::opt<bool> DisableEarlyTailDup("disable-early-taildup", cl::Hidden, | ||
| 57 | cl::desc("Disable pre-register allocation tail duplication")); | 57 | cl::desc("Disable pre-register allocation tail duplication")); | ||
| 58 | static cl::opt<bool> DisableBlockPlacement("disable-block-placement", | 58 | static cl::opt<bool> DisableBlockPlacement("disable-block-placement", | ||
| ▲ Show 20 Lines • Show All 844 Lines • ▼ Show 20 Line(s) | 882 | void TargetPassConfig::addMachineSSAOptimization() { | |||
| 903 | 903 | | |||
| 904 | // Allow targets to insert passes that improve instruction level parallelism, | 904 | // Allow targets to insert passes that improve instruction level parallelism, | ||
| 905 | // like if-conversion. Such passes will typically need dominator trees and | 905 | // like if-conversion. Such passes will typically need dominator trees and | ||
| 906 | // loop info, just like LICM and CSE below. | 906 | // loop info, just like LICM and CSE below. | ||
| 907 | addILPOpts(); | 907 | addILPOpts(); | ||
| 908 | 908 | | |||
| 909 | addPass(&MachineLICMID, false); | 909 | addPass(&MachineLICMID, false); | ||
| 910 | addPass(&MachineCSEID, false); | 910 | addPass(&MachineCSEID, false); | ||
| 911 | 911 | | |||
| 912 | // Coalesce basic blocks with the same branch condition | | |||
| 913 | addPass(&BranchCoalescingID); | | |||
| 914 | | ||||
| 915 | addPass(&MachineSinkingID); | 912 | addPass(&MachineSinkingID); | ||
Not Done ReplyYou don't need this because of the skipFunction query in the pass. echristo: You don't need this because of the skipFunction query in the pass. | |||||
| 916 | 913 | | |||
| 917 | addPass(&PeepholeOptimizerID); | 914 | addPass(&PeepholeOptimizerID); | ||
| 918 | // Clean-up the dead code that may have been generated by peephole | 915 | // Clean-up the dead code that may have been generated by peephole | ||
| 919 | // rewriting. | 916 | // rewriting. | ||
| 920 | addPass(&DeadMachineInstructionElimID); | 917 | addPass(&DeadMachineInstructionElimID); | ||
| 921 | } | 918 | } | ||
| 922 | 919 | | |||
| 923 | //===---------------------------------------------------------------------===// | 920 | //===---------------------------------------------------------------------===// | ||
| ▲ Show 20 Lines • Show All 194 Lines • Show Last 20 Lines | |||||
These seem unrelated. Can you pull them out?