[PATCH] D29200: [JumpThread] Enhance finding partial redundant loads by continuing scanning single predecessor

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 07:38:14 PST 2017

junbuml added inline comments.

Comment at: include/llvm/Analysis/Loads.h:89
+                                bool *IsLoadCSE = nullptr,
+                                unsigned *NumScanedInst = nullptr);
zzheng wrote:
> Can we use 
> ```
> Optional<unsigned &> NumScannedInst
> ```
> here?
Honestly I don't have much idea about the Optional<>, but it looks okay to use it here. However, I believe we can make a separate patch to consider using the Optional for other parameters, not just for this parameter.  Please let me know if we have to use the Optional specifically in this parameter unlikely other parameters.  


More information about the llvm-commits mailing list