[llvm-commits] Deterministic finite automaton based packetizer for VLIW architectures
Jakob Stoklund Olesen
stoklund at 2pi.dk
Wed Nov 30 15:04:40 PST 2011
On Nov 30, 2011, at 2:44 PM, Anshuman Dasgupta wrote:
> I've attached a modified patch with the changes you suggested. A couple of comments: I will remove the emission of the second table when I split up the transition tables for each subtarget in Hexagon since those two tasks are related. Also, it was possible to change most of the STL containers (but not all) to LLVM equivalents. Let me know if it's okay to commit. I will post the CodeGenerator.html changes as a separate patch.
+using namespace llvm;
These are both strictly verboten in headers. Also see http://llvm.org/docs/CodingStandards.html#micro_anonns
Just move these classes into the .cpp file.
+ int CurrentState;
+ const int (*DFAStateInputTable);
Also, please strip trailing whitespace from your patch. We have too much of that already.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits