Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline
Changeset View
Changeset View
Standalone View
Standalone View
include/llvm/CodeGen/Passes.h
Show First 20 Lines • Show All 414 Lines • ▼ Show 20 Lines | /// MachineDominanaceFrontier - This pass is a machine dominators analysis pass. | ||||
/// This pass performs outlining on machine instructions directly before | /// This pass performs outlining on machine instructions directly before | ||||
/// printing assembly. | /// printing assembly. | ||||
ModulePass *createMachineOutlinerPass(); | ModulePass *createMachineOutlinerPass(); | ||||
/// This pass expands the experimental reduction intrinsics into sequences of | /// This pass expands the experimental reduction intrinsics into sequences of | ||||
/// shuffles. | /// shuffles. | ||||
FunctionPass *createExpandReductionsPass(); | FunctionPass *createExpandReductionsPass(); | ||||
/// This pass verifies that outgoing cfa offset and register of predecessor | |||||
/// blocks match incoming cfa offset and register of their successors. | |||||
FunctionPass *createCFIInfoVerifier(); | |||||
/// This pass inserts required CFI instruction at basic block beginning to | |||||
MatzeB: Even though there is a lot of precedent in this file, I'm starting to wonder how useful it is… | |||||
/// correct the CFA calculation rule for that block if necessary. | |||||
FunctionPass *createCFIInstrInserter(); | |||||
} // End llvm namespace | } // End llvm namespace | ||||
#endif | #endif |
Even though there is a lot of precedent in this file, I'm starting to wonder how useful it is to have a description of a pass at the beginning of the passes implementation file and another one in front of the corresponding function in Passes.h. Maybe it would be best to shorten this to:
so we only have to maintain one description of the passes details.