diff options
| author | Jay Foad <jay.foad@amd.com> | 2019-12-19 11:19:36 +0000 |
|---|---|---|
| committer | Jay Foad <jay.foad@amd.com> | 2019-12-19 11:51:52 +0000 |
| commit | c5c935ab6620289ecfd17d46f5653e2dcbd49352 (patch) | |
| tree | e3ff0c943a24985349a251378487454397d22869 /llvm/lib/CodeGen | |
| parent | bbcf1c3496ce2bd1ed87e8fb15ad896e279633ce (diff) | |
| download | bcm5719-llvm-c5c935ab6620289ecfd17d46f5653e2dcbd49352.tar.gz bcm5719-llvm-c5c935ab6620289ecfd17d46f5653e2dcbd49352.zip | |
Make more use of MachineInstr::mayLoadOrStore.
Diffstat (limited to 'llvm/lib/CodeGen')
| -rw-r--r-- | llvm/lib/CodeGen/BranchFolding.cpp | 4 | ||||
| -rw-r--r-- | llvm/lib/CodeGen/ImplicitNullChecks.cpp | 2 | ||||
| -rw-r--r-- | llvm/lib/CodeGen/StackColoring.cpp | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/llvm/lib/CodeGen/BranchFolding.cpp b/llvm/lib/CodeGen/BranchFolding.cpp index 2bf1b392528..4b9c50aeb1d 100644 --- a/llvm/lib/CodeGen/BranchFolding.cpp +++ b/llvm/lib/CodeGen/BranchFolding.cpp @@ -449,7 +449,7 @@ static unsigned EstimateRuntime(MachineBasicBlock::iterator I, continue; if (I->isCall()) Time += 10; - else if (I->mayLoad() || I->mayStore()) + else if (I->mayLoadOrStore()) Time += 2; else ++Time; @@ -835,7 +835,7 @@ mergeOperations(MachineBasicBlock::iterator MBBIStartPos, assert(MBBICommon->isIdenticalTo(*MBBI) && "Expected matching MIIs!"); // Merge MMOs from memory operations in the common block. - if (MBBICommon->mayLoad() || MBBICommon->mayStore()) + if (MBBICommon->mayLoadOrStore()) MBBICommon->cloneMergedMemRefs(*MBB->getParent(), {&*MBBICommon, &*MBBI}); // Drop undef flags if they aren't present in all merged instructions. for (unsigned I = 0, E = MBBICommon->getNumOperands(); I != E; ++I) { diff --git a/llvm/lib/CodeGen/ImplicitNullChecks.cpp b/llvm/lib/CodeGen/ImplicitNullChecks.cpp index b3ca4c1d802..0bbedb0a5ea 100644 --- a/llvm/lib/CodeGen/ImplicitNullChecks.cpp +++ b/llvm/lib/CodeGen/ImplicitNullChecks.cpp @@ -372,7 +372,7 @@ ImplicitNullChecks::isSuitableMemoryOp(const MachineInstr &MI, // We want the mem access to be issued at a sane offset from PointerReg, // so that if PointerReg is null then the access reliably page faults. - if (!((MI.mayLoad() || MI.mayStore()) && !MI.isPredicable() && + if (!(MI.mayLoadOrStore() && !MI.isPredicable() && -PageSize < Offset && Offset < PageSize)) return SR_Unsuitable; diff --git a/llvm/lib/CodeGen/StackColoring.cpp b/llvm/lib/CodeGen/StackColoring.cpp index fc10c7d3a90..b6e81116286 100644 --- a/llvm/lib/CodeGen/StackColoring.cpp +++ b/llvm/lib/CodeGen/StackColoring.cpp @@ -1004,7 +1004,7 @@ void StackColoring::remapInstructions(DenseMap<int, int> &SlotRemap) { // zone are okay, despite the fact that we don't have a good way // for validating all of the usages of the calculation. #ifndef NDEBUG - bool TouchesMemory = I.mayLoad() || I.mayStore(); + bool TouchesMemory = I.mayLoadOrStore(); // If we *don't* protect the user from escaped allocas, don't bother // validating the instructions. if (!I.isDebugInstr() && TouchesMemory && ProtectFromEscapedAllocas) { |
