From 218caa85959a065b6cbfe8962cdd13f472929515 Mon Sep 17 00:00:00 2001 From: Chad Rosier Date: Tue, 26 Jan 2016 19:33:57 +0000 Subject: [PATCH] [ScheduleDAGInstrs] Simplify logic to improve readability. NFC. The call to isInvariantLoad() already returns false for non-load instructions. llvm-svn: 258841 --- lib/CodeGen/ScheduleDAGInstrs.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/CodeGen/ScheduleDAGInstrs.cpp b/lib/CodeGen/ScheduleDAGInstrs.cpp index e0aeb570c2f..00a0b0fc33a 100644 --- a/lib/CodeGen/ScheduleDAGInstrs.cpp +++ b/lib/CodeGen/ScheduleDAGInstrs.cpp @@ -519,8 +519,7 @@ void ScheduleDAGInstrs::addVRegUseDeps(SUnit *SU, unsigned OperIdx) { /// (like a call or something with unmodeled side effects). static inline bool isGlobalMemoryObject(AliasAnalysis *AA, MachineInstr *MI) { return MI->isCall() || MI->hasUnmodeledSideEffects() || - (MI->hasOrderedMemoryRef() && - (!MI->mayLoad() || !MI->isInvariantLoad(AA))); + (MI->hasOrderedMemoryRef() && !MI->isInvariantLoad(AA)); } // This MI might have either incomplete info, or known to be unsafe