[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