[llvm] r334970 - [NFC] make MIFlag accessor functions consistant with usage model
Michael Berg via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 18 11:37:48 PDT 2018
Author: mcberg2017
Date: Mon Jun 18 11:37:48 2018
New Revision: 334970
URL: http://llvm.org/viewvc/llvm-project?rev=334970&view=rev
Log:
[NFC] make MIFlag accessor functions consistant with usage model
Modified:
llvm/trunk/include/llvm/CodeGen/MachineInstr.h
llvm/trunk/lib/CodeGen/MachineInstr.cpp
Modified: llvm/trunk/include/llvm/CodeGen/MachineInstr.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/MachineInstr.h?rev=334970&r1=334969&r2=334970&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/MachineInstr.h (original)
+++ llvm/trunk/include/llvm/CodeGen/MachineInstr.h Mon Jun 18 11:37:48 2018
@@ -189,7 +189,7 @@ public:
}
/// Return the MI flags bitvector.
- uint8_t getFlags() const {
+ uint16_t getFlags() const {
return Flags;
}
@@ -1354,7 +1354,7 @@ public:
/// Return the MIFlags which represent both MachineInstrs. This
/// should be used when merging two MachineInstrs into one. This routine does
/// not modify the MIFlags of this MachineInstr.
- uint8_t mergeFlagsWith(const MachineInstr& Other) const;
+ uint16_t mergeFlagsWith(const MachineInstr& Other) const;
/// Clear this MachineInstr's memory reference descriptor list. This resets
/// the memrefs to their most conservative state. This should be used only
Modified: llvm/trunk/lib/CodeGen/MachineInstr.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineInstr.cpp?rev=334970&r1=334969&r2=334970&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/MachineInstr.cpp (original)
+++ llvm/trunk/lib/CodeGen/MachineInstr.cpp Mon Jun 18 11:37:48 2018
@@ -382,7 +382,7 @@ MachineInstr::mergeMemRefsWith(const Mac
return std::make_pair(MemBegin, CombinedNumMemRefs);
}
-uint8_t MachineInstr::mergeFlagsWith(const MachineInstr &Other) const {
+uint16_t MachineInstr::mergeFlagsWith(const MachineInstr &Other) const {
// For now, the just return the union of the flags. If the flags get more
// complicated over time, we might need more logic here.
return getFlags() | Other.getFlags();
More information about the llvm-commits
mailing list