[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