[llvm] [Instrumentation] Support `MachineFunction` in `OptNoneInstrumentation` (PR #115471)
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 8 12:55:42 PST 2024
https://github.com/aeubanks approved this pull request.
lgtm
for the `isRequired` concept, @chandlerc had the idea to have a `RequiredPassInfoMixin`/`OptionalPassInfoMixin` which had `static constexpr bool isRequired = true/false` that passes could inherit from and error out if there was no `isRequired` on the pass. I like this a lot as opposed to having a default
https://github.com/llvm/llvm-project/pull/115471
More information about the llvm-commits
mailing list