The compiler pass itself isn't an optimization pass though. I'm more curious as of why your service definition's class is a parameter reference and not a class name.

Actually I got this error in in third party bundle. But still if it's possible to define class name as parameter then Compiler pass should handle that.

If this feature is supported, it should probably be done as one of the first things, before all the other compiler passes run.

The right way to solve this would be to resolve the parameter in the pass as we do in

Same concern as @iltar, not sure if we should add support for this, deprecate it in the passes supporting it already or do nothing at all.

@iltar I think we can leave AddSecurityVotersPass as it was before and add new compiler pass of type "optimization" for voters validity check. I did PR for that

Just found a case where the class name is resolved inside the pass:

Yes, it's the same as @chalasr suggests, but if we follow symfony best practices, then I would say that we also should write core to follow it. I mean class name parameters are not recommended and maybe one day it will be deprecated. Having check validity compiler pass doesn't depend on class parameter parsing and it also doesn't change what worked before