[PATCH] D25877: Analysis: Move llvm::getConstantRangeFromMetadata to IR library.

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 12:55:14 PDT 2016


sanjoy accepted this revision.
sanjoy added a comment.
This revision is now accepted and ready to land.

lgtm with nits



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp:18
 #include "llvm/Analysis/Loads.h"
+#include "llvm/IR/ConstantRange.h"
 #include "llvm/IR/DataLayout.h"
----------------
Does this still need to include ValueTracking?


================
Comment at: llvm/lib/Transforms/Scalar/GuardWidening.cpp:48
 #include "llvm/Analysis/PostDominators.h"
 #include "llvm/Analysis/ValueTracking.h"
+#include "llvm/IR/ConstantRange.h"
----------------
Does this still need to include ValueTracking?


https://reviews.llvm.org/D25877





More information about the llvm-commits mailing list