[PATCH] D78734: [CaptureTracking] Make MaxUsesToExplore cheaper (NFC)

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 23 13:34:07 PDT 2020


lebedev.ri added inline comments.


================
Comment at: include/llvm/Analysis/CaptureTracking.h:40
                             bool StoreCaptures,
-                            unsigned MaxUsesToExplore =
-                                getDefaultMaxUsesToExploreForCaptureTracking());
+                            unsigned MaxUsesToExplore = 0);
 
----------------
Have you considered `Optional<unsigned> MaxUsesToExploreOverride`
And if it's not provided, actually override it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78734/new/

https://reviews.llvm.org/D78734





More information about the llvm-commits mailing list