Cleanup cutile-compiler

[ur4t]

No description provided.

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[elibol]

Thanks for the cleanup! Since this touches compiler internals (_function.rs, _type.rs, _value.rs, bounds.rs), I want to look at it more closely. We also have a compiler rewrite in progress that will likely make these changes obsolete. It might make sense to wait until that lands and then take a pass on the updated compiler (if you're interested). Will follow up.

[ur4t]

We also have a compiler rewrite in progress that will likely make these changes obsolete. It might make sense to wait until that lands and then take a pass on the updated compiler.

Sure. I will rebase when it lands.

[ur4t]

I investigated functions marked #[allow(clippy::too_many_arguments)]. Some of them, especially fhmas, require those arguments natively. The others are about compile, from which we can indeed extract shared arguments into an argument "bundle".

I thought #[allow(clippy::too_many_arguments)] might be conspicuous enough so I did not add // TODO.