It's very easy to forget to add `isRequired() { return true; }` for
passes that must be run on all IR, rather than e.g. being skipped on
optnone functions.
This PR adds RequiredPassInfoMixin/OptionalPassInfoMixin which have
`isRequired()`, and the long term vision is there to be no default
`isRequired()` on PassInfoMixin.
However, for out-of-tree migration purposes, we leave a `isRequired() {
return false; }` for now in PassInfoMixin.
https://discourse.llvm.org/t/rfc-introduce-mandatorypassinfomixin-optionalpassinfomixin-for-required-not-required-passes/90547
Assisted-by: Gemini
|
||
|---|---|---|
| .. | ||
| CMakeLists.txt | ||
| LLVMPrintFunctionNames.cpp | ||
| LLVMPrintFunctionNames.exports | ||