Performance

[Anselmo95]

Hello, I have a file containing a package which has 47K lines and it's mostly made of function definition. I know that it is unreasonable and I should split it but probably won't do it any time soon. So I tried to run vsg on it and it takes around 33 minutes with the default configuration.
I was wondering if there was something I could do with the configuration or somewhere else to reduce the runtime.
Just in case here is the output of cProfiler:

And here is the file if someone wants to look at it: prof.zip

Thanks a for this awesome tool !

[jeremiah-c-leary]

Morning @Anselmo95 ,

I'm not surprised a large file takes a long time. The data structure is not really optimized and VSG makes several passes through the file before it starts any analysis. One thing you can do is disable the consistent token rules:

entity_600, procedure_507, subtype_002, variable_011, constant_013, architecture_600, signal_014, alias_declaration_503, type_definition_014, type_definition_501, function_010

These rules take a long time to run.

I think I need to update the data structure to improve performance and to address other issues that are proving difficult to address.

Regards,

--Jeremy

[Anselmo95]

Good evening @jeremiah-c-leary ,

Thanks a lot for your answer, I tried disabling the rules you mentioned but there is not that much of a speedup in the end.
In any case I know that it is kind of unreasonable to have a file of that size, I'll take this opportunity to split it in different files.

As I said before maybe it is useful maybe not but here is the result of the profiler, I also included the config file I'm using now with all the rules you mentioned disabled:
prof.zip

Regards,
Anselmo

[jeremiah-c-leary]

Morning @Anselmo95 ,

I tried to get some information from the vsg.prof file, but it seems the multithreading is masking the data. Could you re-run and add the following option: -p 1?

Thanks,

--Jeremy

[Anselmo95]

Good evening @jeremiah-c-leary ,

Sure, and sorry for the delay on your first message.
Here is the profiler result with the config I used. It's the same as in the other message but none of the rules you mentioned are disabled.
profiler.zip

Thanks for the time,
Anselmo