[llvm-commits] [LLVMdev] [PATCH] Refactoring the DFA generator

Ivan Llopard ivanllopard at gmail.com
Sat Sep 8 01:25:24 PDT 2012


Thanks!

Ivan

On 07/09/2012 23:42, Anshuman Dasgupta wrote:
> Committed in r163424. I made a minor change: renamed StateMap to
> TransitionMap.
>
> -Anshu
>
>
> On 8/29/2012 4:47 PM, Ivan Llopard wrote:
>> On 28/08/2012 00:21, Anshuman Dasgupta wrote:
>>> 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.
>>
>> Sure. Please, take a look at the new patch.
>>
>> Ivan
>




More information about the llvm-commits mailing list