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 /mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp | |
| parent | 0ca80eb5e814a6d061556888c9c9fbd04f054a80 (diff) | |
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 'mlir/test/lib/Dialect/Vector/TestVectorTransforms.cpp')
0 files changed, 0 insertions, 0 deletions