Fix/parser improvements

[duckdoom5]

• Reserve some memory to speedup the process
• Link opts->verbose and opts->skipFunctionBodies to clang
• Assign some properties that were not assigned

[tritao]

Was ready to hit the merge button, except it doesn't pass CI 😄

[duckdoom5]

I'll check it out. See if I can figure out what makes it crash.
I think it's just crashing in some project I didn't test. My guess is this line needs to be added

[duckdoom5]

Okay, so I fixed the two errors that popped up in test builds.
The first one seems to be caused by symbols now being generated for template args.
That was a simple fix.

Can't return T(); if T is a const reference type.

However the second error I'm thoroughly confused by. But I have one theory..
The issue was that the symbols code generator (see ImplementIfAbstract) would implement the function virtual ~PureImplementedDtor() = 0; (marked pure virtual)
as PureImplementedDtor::~PureImplementedDtor() {}

Which was already defined in code leading to an error.
I would argue implementing a function marked pure virtual anyway, is a bug, but let me know if you think it should be solved by looping through the available methods.

So my theory is this:
I noticed that when passing opts->skipFunctionBodies to ParseAST some elements in the AST got ordered differently. I'm guessing that previously, the destructor was parsed before the default constructor?

Edit: There seems to be a third error when building a test. Now somehow the FunctionTemplate function is missing 😅
Edit 2: Ah this must be why. It's missing explicit instantiations then.

[tritao]

Is this one ready to go?

[duckdoom5]

Yes it is :)