[all-commits] [llvm/llvm-project] 2b2827: [CaptureTracking] Make MaxUsesToExplore cheaper (NFC)
Nikita Popov via All-commits
all-commits at lists.llvm.org
Sun Apr 26 00:57:10 PDT 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: 2b2827552ad7e0fbd30597d714e751516f83e0f9
https://github.com/llvm/llvm-project/commit/2b2827552ad7e0fbd30597d714e751516f83e0f9
Author: Nikita Popov <nikita.ppv at gmail.com>
Date: 2020-04-26 (Sun, 26 Apr 2020)
Changed paths:
M llvm/include/llvm/Analysis/CaptureTracking.h
M llvm/lib/Analysis/CaptureTracking.cpp
Log Message:
-----------
[CaptureTracking] Make MaxUsesToExplore cheaper (NFC)
The change in D78624 had a noticeable negative compile-time impact.
It seems that going through a function call for the MaxUsesToExplore
default is fairly expensive, at least if LLVM is not built with LTO.
This patch makes MaxUsesToExpore default to 0 and assigns the actual
default in the implementation instead. This recovers most of the
regression.
Differential Revision: https://reviews.llvm.org/D78734
More information about the All-commits
mailing list