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
25 lines
834 B
C++
25 lines
834 B
C++
//===- llvm/CodeGen/FixupStatepointCallerSaved.h ----------------*- C++ -*-===//
|
|
//
|
|
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
|
|
// See https://llvm.org/LICENSE.txt for license information.
|
|
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
#ifndef LLVM_CODEGEN_FIXUPSTATEPOINTCALLERSAVED_H
|
|
#define LLVM_CODEGEN_FIXUPSTATEPOINTCALLERSAVED_H
|
|
|
|
#include "llvm/CodeGen/MachinePassManager.h"
|
|
|
|
namespace llvm {
|
|
|
|
class FixupStatepointCallerSavedPass
|
|
: public OptionalPassInfoMixin<FixupStatepointCallerSavedPass> {
|
|
public:
|
|
PreservedAnalyses run(MachineFunction &MF,
|
|
MachineFunctionAnalysisManager &MFAM);
|
|
};
|
|
|
|
} // namespace llvm
|
|
|
|
#endif // LLVM_CODEGEN_FIXUPSTATEPOINTCALLERSAVED_H
|