[llvm] 3ff6ed8 - [LiveDebugValues] Use std::optional in InstrRefBasedImpl.cpp (NFC)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Sat Nov 26 14:52:38 PST 2022
Author: Kazu Hirata
Date: 2022-11-26T14:52:33-08:00
New Revision: 3ff6ed810381846c37be960e13d7f71596226a3f
URL: https://github.com/llvm/llvm-project/commit/3ff6ed810381846c37be960e13d7f71596226a3f
DIFF: https://github.com/llvm/llvm-project/commit/3ff6ed810381846c37be960e13d7f71596226a3f.diff
LOG: [LiveDebugValues] Use std::optional in InstrRefBasedImpl.cpp (NFC)
This is part of an effort to migrate from llvm::Optional to
std::optional:
https://discourse.llvm.org/t/deprecating-llvm-optional-x-hasvalue-getvalue-getvalueor/63716
Added:
Modified:
llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
Removed:
################################################################################
diff --git a/llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp b/llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
index f6a55db6a4232..11bb3eedd78c0 100644
--- a/llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
+++ b/llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.cpp
@@ -127,6 +127,7 @@
#include "InstrRefBasedImpl.h"
#include "LiveDebugValues.h"
+#include <optional>
using namespace llvm;
using namespace LiveDebugValues;
@@ -740,7 +741,7 @@ class TransferTracker {
// Examine the remaining variable locations: if we can find the same value
// again, we can recover the location.
- Optional<LocIdx> NewLoc;
+ std::optional<LocIdx> NewLoc;
for (auto Loc : MTracker->locations())
if (Loc.Value == OldValue)
NewLoc = Loc.Idx;
@@ -1560,7 +1561,7 @@ bool InstrRefBasedLDV::transferDebugInstrRef(MachineInstr &MI,
// Pick a location for the machine value number, if such a location exists.
// (This information could be stored in TransferTracker to make it faster).
- Optional<LocIdx> FoundLoc;
+ std::optional<LocIdx> FoundLoc;
for (auto Location : MTracker->locations()) {
LocIdx CurL = Location.Idx;
ValueIDNum ID = MTracker->readMLoc(CurL);
@@ -1913,7 +1914,7 @@ bool InstrRefBasedLDV::transferSpillOrRestoreInst(MachineInstr &MI) {
// continue past this store.
for (unsigned SlotIdx = 0; SlotIdx < MTracker->NumSlotIdxes; ++SlotIdx) {
unsigned SpillID = MTracker->getSpillIDWithIdx(*Loc, SlotIdx);
- Optional<LocIdx> MLoc = MTracker->getSpillMLoc(SpillID);
+ std::optional<LocIdx> MLoc = MTracker->getSpillMLoc(SpillID);
if (!MLoc)
continue;
More information about the llvm-commits
mailing list