[PATCH] D42270: [ValueLattice] Use getters instead of direct accesses (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 19 02:36:30 PST 2018


This revision was automatically updated to reflect the committed changes.
Closed by commit rL322933: [ValueLattice] Use getters instead of direct accesses (NFC). (authored by fhahn, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D42270?vs=130514&id=130575#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D42270

Files:
  llvm/trunk/include/llvm/Analysis/ValueLattice.h


Index: llvm/trunk/include/llvm/Analysis/ValueLattice.h
===================================================================
--- llvm/trunk/include/llvm/Analysis/ValueLattice.h
+++ llvm/trunk/include/llvm/Analysis/ValueLattice.h
@@ -189,14 +189,14 @@
     }
 
     if (isConstant()) {
-      if (RHS.isConstant() && Val == RHS.Val)
+      if (RHS.isConstant() && getConstant() == RHS.getConstant())
         return false;
       markOverdefined();
       return true;
     }
 
     if (isNotConstant()) {
-      if (RHS.isNotConstant() && Val == RHS.Val)
+      if (RHS.isNotConstant() && getNotConstant() == RHS.getNotConstant())
         return false;
       markOverdefined();
       return true;
@@ -209,7 +209,7 @@
       markOverdefined();
       return true;
     }
-    ConstantRange NewR = Range.unionWith(RHS.getConstantRange());
+    ConstantRange NewR = getConstantRange().unionWith(RHS.getConstantRange());
     if (NewR.isFullSet())
       markOverdefined();
     else


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42270.130575.patch
Type: text/x-patch
Size: 981 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180119/b6da00ee/attachment.bin>


More information about the llvm-commits mailing list