@VSadov - any context on why this is commented instead of deleted?

At some point we had internal visitors generic visitors parameterized on parameters like SyntaxVisitor<TArgument, TResult> .They were convenient, but not very fast because of virtual generic Accept. Eventually all the visitors switched to the faster version SyntaxVisitor<TResult>.
There were some arguments that at some point we might still need the convenience of SyntaxVisitor<TArgument, TResult>, so it was not removed, just commented out.

Perhaps it is time to remove it :-)

.

@stefansedich If you haven't already, can you please sign the CLA here - https://cla2.dotnetfoundation.org ?

@stefansedich, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, DNFBOT;

All done @srivatsn

Great thanks.