Existing clients are converted to use MachineModuleInfoWrapperPass. The new interface is for defining a new pass manager API in CodeGen (D64179).
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 34545 Build 34544: arc lint + arc unit
Event Timeline
Comment Actions
Please, add a meaningful commit message that describes in high-level terms what/why you are doing here.
Comment Actions
Hello, Charlie went back to school after his internship, I'll commit this on his behalf. Please let me know if there are any concerns.
llvm/include/llvm/CodeGen/MachineModuleInfo.h | ||
---|---|---|
291 | {}; -> {} |
llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h | ||
---|---|---|
151 ↗ | (On Diff #222461) | This causes ../../llvm/include/llvm/CodeGen/MachineModuleInfo.h:151:22: warning: explicitly defaulted move assignment operator is implicitly deleted [-Wdefaulted-function-deleted] MachineModuleInfo &operator=(MachineModuleInfo &&MMII) = default; ^ ../../llvm/include/llvm/CodeGen/MachineModuleInfo.h:82:28: note: move assignment operator of 'MachineModuleInfo' is implicitly deleted because field 'TM' is of reference type 'const llvm::LLVMTargetMachine &' const LLVMTargetMachine &TM; ^ |
llvm/trunk/include/llvm/CodeGen/MachineModuleInfo.h | ||
---|---|---|
268 ↗ | (On Diff #222461) | (reposting from the phab commit, since it looks like phab didn't send to any mailing lists...) Is this method actually defined anywhere? I tried migrating an out of tree use of MachineModuleInfo to wrap this but I got a link error, and I don't think this is defined. At any rate, I think I can just use the constructor above instead. |
{}; -> {}