use space star instead of star space
authorSebastian Pop <spop@codeaurora.org>
Tue, 6 Dec 2011 17:34:16 +0000 (17:34 +0000)
committerSebastian Pop <spop@codeaurora.org>
Tue, 6 Dec 2011 17:34:16 +0000 (17:34 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@145944 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/CodeGen/DFAPacketizer.h
lib/CodeGen/DFAPacketizer.cpp
utils/TableGen/DFAPacketizerEmitter.cpp
utils/TableGen/DFAPacketizerEmitter.h
utils/TableGen/SubtargetEmitter.cpp

index 933e6e0cc4b5894fc56d5b127772a4f3e6734ee8..0a11e0cfe42a979d2914060c851004e13e39fd1e 100644 (file)
@@ -49,8 +49,8 @@ private:
   void ReadTable(unsigned int state);
 
 public:
-  DFAPacketizer(const InstrItineraryDataI, const int (*SIT)[2],
-                const unsignedSET);
+  DFAPacketizer(const InstrItineraryData *I, const int (*SIT)[2],
+                const unsigned *SET);
 
   // Reset the current state to make all resources available.
   void clearResources() {
@@ -59,19 +59,19 @@ public:
 
   // canReserveResources - Check if the resources occupied by a MCInstrDesc
   // are available in the current state.
-  bool canReserveResources(const llvm::MCInstrDescMID);
+  bool canReserveResources(const llvm::MCInstrDesc *MID);
 
   // reserveResources - Reserve the resources occupied by a MCInstrDesc and
   // change the current state to reflect that change.
-  void reserveResources(const llvm::MCInstrDescMID);
+  void reserveResources(const llvm::MCInstrDesc *MID);
 
   // canReserveResources - Check if the resources occupied by a machine
   // instruction are available in the current state.
-  bool canReserveResources(llvm::MachineInstrMI);
+  bool canReserveResources(llvm::MachineInstr *MI);
 
   // reserveResources - Reserve the resources occupied by a machine
   // instruction and change the current state to reflect that change.
-  void reserveResources(llvm::MachineInstrMI);
+  void reserveResources(llvm::MachineInstr *MI);
 };
 }
 
index c0e505f0d88545b66e6568c47519df6f4320e322..16276bda3762ac468dfad80f54bed96c29dc1eec 100644 (file)
@@ -29,7 +29,7 @@
 using namespace llvm;
 
 DFAPacketizer::DFAPacketizer(const InstrItineraryData *I, const int (*SIT)[2],
-                             const unsignedSET):
+                             const unsigned *SET):
   InstrItins(I), CurrentState(0), DFAStateInputTable(SIT),
   DFAStateEntryTable(SET) {}
 
@@ -60,9 +60,9 @@ void DFAPacketizer::ReadTable(unsigned int state) {
 
 // canReserveResources - Check if the resources occupied by a MCInstrDesc
 // are available in the current state.
-bool DFAPacketizer::canReserveResources(const llvm::MCInstrDescMID) {
+bool DFAPacketizer::canReserveResources(const llvm::MCInstrDesc *MID) {
   unsigned InsnClass = MID->getSchedClass();
-  const llvm::InstrStageIS = InstrItins->beginStage(InsnClass);
+  const llvm::InstrStage *IS = InstrItins->beginStage(InsnClass);
   unsigned FuncUnits = IS->getUnits();
   UnsignPair StateTrans = UnsignPair(CurrentState, FuncUnits);
   ReadTable(CurrentState);
@@ -72,9 +72,9 @@ bool DFAPacketizer::canReserveResources(const llvm::MCInstrDesc* MID) {
 
 // reserveResources - Reserve the resources occupied by a MCInstrDesc and
 // change the current state to reflect that change.
-void DFAPacketizer::reserveResources(const llvm::MCInstrDescMID) {
+void DFAPacketizer::reserveResources(const llvm::MCInstrDesc *MID) {
   unsigned InsnClass = MID->getSchedClass();
-  const llvm::InstrStageIS = InstrItins->beginStage(InsnClass);
+  const llvm::InstrStage *IS = InstrItins->beginStage(InsnClass);
   unsigned FuncUnits = IS->getUnits();
   UnsignPair StateTrans = UnsignPair(CurrentState, FuncUnits);
   ReadTable(CurrentState);
@@ -85,14 +85,14 @@ void DFAPacketizer::reserveResources(const llvm::MCInstrDesc* MID) {
 
 // canReserveResources - Check if the resources occupied by a machine
 // instruction are available in the current state.
-bool DFAPacketizer::canReserveResources(llvm::MachineInstrMI) {
-  const llvm::MCInstrDescMID = MI->getDesc();
+bool DFAPacketizer::canReserveResources(llvm::MachineInstr *MI) {
+  const llvm::MCInstrDesc &MID = MI->getDesc();
   return canReserveResources(&MID);
 }
 
 // reserveResources - Reserve the resources occupied by a machine
 // instruction and change the current state to reflect that change.
-void DFAPacketizer::reserveResources(llvm::MachineInstrMI) {
-  const llvm::MCInstrDescMID = MI->getDesc();
+void DFAPacketizer::reserveResources(llvm::MachineInstr *MI) {
+  const llvm::MCInstrDesc &MID = MI->getDesc();
   reserveResources(&MID);
 }
index 32bb26ec41b3a21786b5dca4e9ec1645614089b0..2862d0cefc14147cafe07dd2c9a867c4af1bef62 100644 (file)
@@ -53,7 +53,7 @@ class State {
   std::set<unsigned> stateInfo;
 
   State();
-  State(const StateS);
+  State(const State &S);
 
   //
   // canAddInsnClass - Returns true if an instruction of type InsnClass is a
@@ -63,7 +63,7 @@ class State {
   // PossibleStates is the set of valid resource states that ensue from valid
   // transitions.
   //
-  bool canAddInsnClass(unsigned InsnClass, std::set<unsigned>PossibleStates);
+  bool canAddInsnClass(unsigned InsnClass, std::set<unsigned> &PossibleStates);
 };
 } // End anonymous namespace.
 
@@ -73,11 +73,11 @@ struct Transition {
  public:
   static int currentTransitionNum;
   int transitionNum;
-  Statefrom;
+  State *from;
   unsigned input;
-  Stateto;
+  State *to;
 
-  Transition(State* from_, unsigned input_, State* to_);
+  Transition(State *from_, unsigned input_, State *to_);
 };
 } // End anonymous namespace.
 
@@ -87,7 +87,7 @@ struct Transition {
 //
 namespace {
 struct ltState {
-  bool operator()(const State* s1, const State* s2) const;
+  bool operator()(const State *s1, const State *s2) const;
 };
 } // End anonymous namespace.
 
@@ -105,7 +105,7 @@ public:
 
   // Map from a state to the list of transitions with that state as source.
   std::map<State*, SmallVector<Transition*, 16>, ltState> stateTransitions;
-  StatecurrentState;
+  State *currentState;
 
   // Highest valued Input seen.
   unsigned LargestInput;
@@ -114,25 +114,25 @@ public:
   // Modify the DFA.
   //
   void initialize();
-  void addState(State*);
-  void addTransition(Transition*);
+  void addState(State *);
+  void addTransition(Transition *);
 
   //
   // getTransition -  Return the state when a transition is made from
   // State From with Input I. If a transition is not found, return NULL.
   //
-  State* getTransition(State*, unsigned);
+  State *getTransition(State *, unsigned);
 
   //
   // isValidTransition: Predicate that checks if there is a valid transition
   // from state From on input InsnClass.
   //
-  bool isValidTransition(StateFrom, unsigned InsnClass);
+  bool isValidTransition(State *From, unsigned InsnClass);
 
   //
   // writeTable: Print out a table representing the DFA.
   //
-  void writeTableAndAPI(raw_ostream &OS, const std::stringClassName);
+  void writeTableAndAPI(raw_ostream &OS, const std::string &ClassName);
 };
 } // End anonymous namespace.
 
@@ -144,12 +144,12 @@ State::State() :
   stateNum(currentStateNum++), isInitial(false) {}
 
 
-State::State(const StateS) :
+State::State(const State &S) :
   stateNum(currentStateNum++), isInitial(S.isInitial),
   stateInfo(S.stateInfo) {}
 
 
-Transition::Transition(State* from_, unsigned input_, State* to_) :
+Transition::Transition(State *from_, unsigned input_, State *to_) :
   transitionNum(currentTransitionNum++), from(from_), input(input_),
   to(to_) {}
 
@@ -158,7 +158,7 @@ DFA::DFA() :
   LargestInput(0) {}
 
 
-bool ltState::operator()(const State* s1, const State* s2) const {
+bool ltState::operator()(const State *s1, const State *s2) const {
     return (s1->stateNum < s2->stateNum);
 }
 
@@ -172,7 +172,7 @@ bool ltState::operator()(const State* s1, const State* s2) const {
 // transitions.
 //
 bool State::canAddInsnClass(unsigned InsnClass,
-                            std::set<unsigned>PossibleStates) {
+                            std::set<unsigned> &PossibleStates) {
   //
   // Iterate over all resource states in currentState.
   //
@@ -224,13 +224,13 @@ void DFA::initialize() {
 }
 
 
-void DFA::addState(StateS) {
+void DFA::addState(State *S) {
   assert(!states.count(S) && "State already exists");
   states.insert(S);
 }
 
 
-void DFA::addTransition(TransitionT) {
+void DFA::addTransition(Transition *T) {
   // Update LargestInput.
   if (T->input > LargestInput)
     LargestInput = T->input;
@@ -244,7 +244,7 @@ void DFA::addTransition(Transition* T) {
 // getTransition - Return the state when a transition is made from
 // State From with Input I. If a transition is not found, return NULL.
 //
-State* DFA::getTransition(State* From, unsigned I) {
+State *DFA::getTransition(State *From, unsigned I) {
   // Do we have a transition from state From?
   if (!stateTransitions.count(From))
     return NULL;
@@ -260,7 +260,7 @@ State* DFA::getTransition(State* From, unsigned I) {
 }
 
 
-bool DFA::isValidTransition(StateFrom, unsigned InsnClass) {
+bool DFA::isValidTransition(State *From, unsigned InsnClass) {
   return (getTransition(From, InsnClass) != NULL);
 }
 
@@ -268,7 +268,7 @@ bool DFA::isValidTransition(State* From, unsigned InsnClass) {
 int State::currentStateNum = 0;
 int Transition::currentTransitionNum = 0;
 
-DFAGen::DFAGen(RecordKeeperR):
+DFAGen::DFAGen(RecordKeeper &R):
   TargetName(CodeGenTarget(R).getName()),
   allInsnClasses(), Records(R) {}
 
@@ -284,7 +284,7 @@ DFAGen::DFAGen(RecordKeeper& R):
 //                         the ith state.
 //
 //
-void DFA::writeTableAndAPI(raw_ostream &OS, const std::stringTargetName) {
+void DFA::writeTableAndAPI(raw_ostream &OS, const std::string &TargetName) {
   std::set<State*, ltState>::iterator SI = states.begin();
   // This table provides a map to the beginning of the transitions for State s
   // in DFAStateInputTable.
@@ -334,7 +334,7 @@ void DFA::writeTableAndAPI(raw_ostream &OS, const std::string& TargetName) {
   std::string SubTargetClassName = TargetName + "GenSubtargetInfo";
   OS << "\n" << "#include \"llvm/CodeGen/DFAPacketizer.h\"\n";
   OS << "namespace llvm {\n";
-  OS << "DFAPacketizer" << SubTargetClassName << "::"
+  OS << "DFAPacketizer *" << SubTargetClassName << "::"
      << "createDFAPacketizer(const InstrItineraryData *IID) const {\n"
      << "   return new DFAPacketizer(IID, " << TargetName
      << "DFAStateInputTable, " << TargetName << "DFAStateEntryTable);\n}\n\n";
@@ -444,7 +444,7 @@ void DFAGen::run(raw_ostream &OS) {
   // Run a worklist algorithm to generate the DFA.
   //
   DFA D;
-  StateInitial = new State;
+  State *Initial = new State;
   Initial->isInitial = true;
   Initial->stateInfo.insert(0x0);
   D.addState(Initial);
@@ -471,7 +471,7 @@ void DFAGen::run(raw_ostream &OS) {
   //             Add S' to Visited
   //
   while (!WorkList.empty()) {
-    Statecurrent = WorkList.pop_back_val();
+    State *current = WorkList.pop_back_val();
     for (DenseSet<unsigned>::iterator CI = allInsnClasses.begin(),
            CE = allInsnClasses.end(); CI != CE; ++CI) {
       unsigned InsnClass = *CI;
@@ -483,7 +483,7 @@ void DFAGen::run(raw_ostream &OS) {
       //
       if (!D.getTransition(current, InsnClass) &&
           current->canAddInsnClass(InsnClass, NewStateResources)) {
-        StateNewState = NULL;
+        State *NewState = NULL;
 
         //
         // If we have seen this state before, then do not create a new state.
@@ -500,7 +500,7 @@ void DFAGen::run(raw_ostream &OS) {
           WorkList.push_back(NewState);
         }
 
-        TransitionNewTransition = new Transition(current, InsnClass,
+        Transition *NewTransition = new Transition(current, InsnClass,
                                                    NewState);
         D.addTransition(NewTransition);
       }
index 9a13e78c5deab9fcd394f3ae178695e6cf166c00..8f47dded322ae4b7fc8d98d138f3b55b637d3bb1 100644 (file)
@@ -38,7 +38,7 @@ private:
   RecordKeeper &Records;
 
 public:
-  DFAGen(RecordKeeperR);
+  DFAGen(RecordKeeper &R);
 
   //
   // collectAllInsnClasses: Populate allInsnClasses which is a set of units
index 33f4377114a2b326d46936ea7197b3f6e9fcdbf3..34cd9a9e906d3f9d50bdda6bccff509d282ea524 100644 (file)
@@ -716,7 +716,7 @@ void SubtargetEmitter::run(raw_ostream &OS) {
      << "  explicit " << ClassName << "(StringRef TT, StringRef CPU, "
      << "StringRef FS);\n"
      << "public:\n"
-     << "  DFAPacketizer* createDFAPacketizer(const InstrItineraryData* IID)"
+     << "  DFAPacketizer *createDFAPacketizer(const InstrItineraryData *IID)"
      << " const;\n"
      << "};\n";
   OS << "} // End llvm namespace \n";