[PATCH] D16231: isSafeToLoadUnconditionally support queries without a context

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 18:21:00 PST 2016


hfinkel added inline comments.

================
Comment at: include/llvm/Analysis/Loads.h:37
@@ -31,2 +36,3 @@
                                  Instruction *ScanFrom,
+                                 const DataLayout &DL,
                                  const DominatorTree *DT = nullptr,
----------------
If DL is required, and ScanFrom is now not required, why not flip the order so that you can default ScanFrom to nullptr?


http://reviews.llvm.org/D16231





More information about the llvm-commits mailing list