[all-commits] [llvm/llvm-project] 9ace4b: Revert "[SCEV] Factor out part of wrap flag detect...

Nikita Popov via All-commits all-commits at lists.llvm.org
Sun Nov 15 01:27:57 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 9ace4b337fe26dd2c15da854767064acdd8da543
      https://github.com/llvm/llvm-project/commit/9ace4b337fe26dd2c15da854767064acdd8da543
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-11-15 (Sun, 15 Nov 2020)

  Changed paths:
    M llvm/include/llvm/Analysis/ScalarEvolution.h
    M llvm/lib/Analysis/ScalarEvolution.cpp
    M llvm/test/Analysis/ScalarEvolution/pr22641.ll
    M llvm/test/Analysis/ScalarEvolution/sext-iv-2.ll
    M llvm/test/Transforms/IndVarSimplify/X86/loop-invariant-conditions.ll

  Log Message:
  -----------
  Revert "[SCEV] Factor out part of wrap flag detection logic [NFC-ish]"

This reverts commit 1ec6e1eb8a084bffae8a40236eb9925d8026dd07.

This change causes a significant compile-time regression:
https://llvm-compile-time-tracker.com/compare.php?from=dd0b8b94d0796bd895cc998dd163b4fbebceb0b8&to=1ec6e1eb8a084bffae8a40236eb9925d8026dd07&stat=instructions

I assume that this is due to the non-NFC part of the change, which
now performs expensive nowrap inference even for nowrap flags that
are not used by the particular code.




More information about the All-commits mailing list