diff options
| author | Mircea Trofin <mtrofin@google.com> | 2020-01-15 13:33:58 -0800 |
|---|---|---|
| committer | Mircea Trofin <mtrofin@google.com> | 2020-01-15 13:34:20 -0800 |
| commit | 5466597fee379b44f643cee0e0632fdef8fb6b21 (patch) | |
| tree | 3f1704c13dc46084ad012ab788efa56416ef7eeb /llvm/lib/CodeGen/MachineScheduler.cpp | |
| parent | fa26a37d36991dd19276c2112ae03e962debe0e4 (diff) | |
| download | llvm-5466597fee379b44f643cee0e0632fdef8fb6b21.zip llvm-5466597fee379b44f643cee0e0632fdef8fb6b21.tar.gz llvm-5466597fee379b44f643cee0e0632fdef8fb6b21.tar.bz2 | |
[NFC] Refactor InlineResult for readability
Summary:
InlineResult is used both in APIs assessing whether a call site is
inlinable (e.g. llvm::isInlineViable) as well as in the function
inlining utility (llvm::InlineFunction). It means slightly different
things (can/should inlining happen, vs did it happen), and the
implicit casting may introduce ambiguity (casting from 'false' in
InlineFunction will default a message about hight costs,
which is incorrect here).
The change renames the type to a more generic name, and disables
implicit constructors.
Reviewers: eraman, davidxl
Reviewed By: davidxl
Subscribers: kerbowa, arsenm, jvesely, nhaehnle, eraman, hiraditya, haicheng, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D72744
Diffstat (limited to 'llvm/lib/CodeGen/MachineScheduler.cpp')
0 files changed, 0 insertions, 0 deletions