diff options
author | Ramkumar Ramachandra <Ramkumar.Ramachandra@imgtec.com> | 2023-12-05 08:18:17 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-05 08:18:17 +0000 |
commit | e8dbe945f39f2249fe24e0d62ec8ac998e853c2b (patch) | |
tree | 44f1265328c998b2b96b8b1025c1dded48fcf36c /flang/lib/Frontend/CompilerInvocation.cpp | |
parent | 0ca80eb5e814a6d061556888c9c9fbd04f054a80 (diff) | |
download | llvm-e8dbe945f39f2249fe24e0d62ec8ac998e853c2b.zip llvm-e8dbe945f39f2249fe24e0d62ec8ac998e853c2b.tar.gz llvm-e8dbe945f39f2249fe24e0d62ec8ac998e853c2b.tar.bz2 |
TargetInstrInfo, TargetSchedule: fix non-NFC parts of 9468de4 (#74338)
Follow up on a post-commit review of 9468de4 (TargetInstrInfo: make
getOperandLatency return optional (NFC)) by Bjorn Pettersson to fix a
couple of things that are not NFC:
- std::optional<T>::operator<= returns true if the first operand is a
std::nullopt and second operand is T. Fix a couple of places where we
assumed it would return false.
- In TargetSchedule, computeInstrCost could take another codepath,
returning InstrLatency instead of DefaultDefLatency. Fix one instance
not accounting for this behavior.
Diffstat (limited to 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions