search for: dfapacketizeremitter

...es for a given InsnClass - AddInsnClass() performs the actual computation of possible states. I've regenerated the DFA table of Hexagon and all seems to be ok. What do you think about these changes ? Ivan -------------- next part -------------- A non-text attachment was scrubbed... Name: DFAPacketizerEmitter.speedup.patch Type: text/x-patch Size: 4957 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120614/93dca92e/attachment.bin>

Committed in r159281. -Anshu On 6/26/2012 3:04 AM, Ivan Llopard wrote: > Hi Anshu, > > I don't have commit access. It applies correctly on trunk, I've just > checked it. Could you please commit it? > > Ivan > > On 26/06/2012 04:44, adasgupt at codeaurora.org wrote: >> Hi Ivan, >> >> Sorry, I should have been more explicit in my last email. The

Hi Anshu, I don't have commit access. It applies correctly on trunk, I've just checked it. Could you please commit it? Ivan On 26/06/2012 04:44, adasgupt at codeaurora.org wrote: > Hi Ivan, > > Sorry, I should have been more explicit in my last email. The patch looks > good to me. Please check that it applies on trunk and go ahead and commit. > > Thanks > -Anshu

...> llvm-commits mailing list >>>>>>>> llvm-commits at cs.uiuc.edu >>>>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits >>>>>>> >>>>> >>> > > -------------- next part -------------- Index: DFAPacketizerEmitter.cpp =================================================================== --- DFAPacketizerEmitter.cpp (révision 159340) +++ DFAPacketizerEmitter.cpp (copie de travail) @@ -76,15 +76,19 @@ // // namespace { +struct Transition; + class State { public: static int currentStateNum; int stateN...

...eFiles/obj.llvm-tblgen.dir/DAGISelEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/GlobalISelEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/InfoByHwMode.cpp.o CMakeFiles/obj.llv...

...eFiles/obj.llvm-tblgen.dir/DAGISelEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/GlobalISelEmitter.cpp.o CMakeFiles/obj.llvm-tblgen.dir/InfoByHwMode.cpp.o CMakeFiles/obj.llv...

....o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/GlobalISelEmitter.cpp.o >> CMakeFiles/obj.llvm-tbl...

....o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >> CMakeFiles/obj.llvm-tblgen.dir/GlobalISelEmitter.cpp.o >> CMakeFiles/obj.llvm-tbl...

...CMakeFiles\llvm-tblgen.dir\DAGISelMatcherEmitter.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\DAGISelMatcherGen.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\DAGISelMatcherOpt.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\DAGISelMatcher.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\DFAPacketizerEmitter.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\DisassemblerEmitter.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\FastISelEmitter.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\FixedLenDecoderEmitter.cpp.obj utils\TableGen\CMakeFiles\llvm-tblgen.dir\GlobalISelEmitter.cpp.obj utils\TableGen...

...en.dir/DAGISelMatcherEmitter.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >> >> CMakeFiles/obj.llvm-tblgen.dir/GlobalISelEmitter.cp...

...tblgen.dir/DAGISelMatcherEmitter.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >>>> CMakeFiles/obj.llvm-tblgen.dir/GlobalISelEmitter.cpp.o...

...gt;> >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >> >> >> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >> >> >> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >> >> >> CMakeFiles/obj....

....cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >>>>>> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >>>>>> CMakeFiles/obj.llvm-tbl...

...s attached but if you prefer (2) I can remake it. I failed to use SmallSet to store the transitions in State because I needed to iterate on it. What kind of llvm structure may I use? Ivan > > -Anshu > -------------- next part -------------- A non-text attachment was scrubbed... Name: DFAPacketizerEmitter.new.patch Type: text/x-patch Size: 8628 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120825/2ec5f657/attachment.bin>

...MakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherGen.cpp.o >> >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcherOpt.cpp.o >> >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DAGISelMatcher.cpp.o >> >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DFAPacketizerEmitter.cpp.o >> >> >> >> CMakeFiles/obj.llvm-tblgen.dir/DisassemblerEmitter.cpp.o >> >> >> >> CMakeFiles/obj.llvm-tblgen.dir/FastISelEmitter.cpp.o >> >> >> >> CMakeFiles/obj.llvm-tblgen.dir/FixedLenDecoderEmitter.cpp.o >> >&g...

Ivan, Thanks for working on the DFA generator. I'll take a look at the changes in detail but from your description, I like the general nature of the modifications. -Anshu --- Qualcomm Innovation Center, Inc is a member of the Code Aurora Forum On 6/14/2012 8:22 AM, Ivan Llopard wrote: > Hi, > > I've refactored the DFA generator in TableGen because it takes too > much

Hi Ivan, > I missed last 2 commits made by Alexey. Following his advices, I > updated the patch. It should be ok now. > Thanks Anshu! > > I've recently added more functional units to our Schedule.td and the > generation time became painfully long. In fact, the main problem was > in writeTableAndAPI(). I propose another patch to fix it: > - Fixed memory leaks. > -

Hi Ivan, The patch looks good to me. I have a couple of minor comments: +void State::AddInsnClass(unsigned InsnClass, Add a top level comment describing the function + std::map<State*, std::set<Transition*, ltTransition>, ltState> stateTransitions; You should be able to use SmallSet here. Also, this line exceeds 80 columns. On a related note, is the CachedTable mechanism in

Ivan, Thanks for working on the patch. It looks good to me except for the removal of the Transition class: > (1) Should I completely remove Transition and create a map structure in State (input, state) to replace them? Yes, please remove the Transition class and create a map structure in State instead of TransitionSet. Thanks -Anshu On 8/25/2012 6:42 AM, Ivan Llopard wrote: > Hi

...store the transitions in State because I >> needed to iterate on it. What kind of llvm structure may I use? >> >> Ivan >> >>> >>> -Anshu >>> >> >> > -------------- next part -------------- A non-text attachment was scrubbed... Name: DFAPacketizerEmitter.new.patch Type: text/x-patch Size: 8117 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120829/4a66c373/attachment.bin>