[all-commits] [llvm/llvm-project] 2d0d42: [InstCombine] Add test for -expensive-combines opt...

[Nikita Popov via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 2d0d4235a282e0f900d31ac1054aafc0c526245c
      https://github.com/llvm/llvm-project/commit/2d0d4235a282e0f900d31ac1054aafc0c526245c
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-01-17 (Fri, 17 Jan 2020)

  Changed paths:
    A llvm/test/Transforms/InstCombine/expensive-combines.ll

  Log Message:
  -----------
  [InstCombine] Add test for -expensive-combines option; NFC

This shows that -expensive-combines=0 is ignored.


  Commit: 2ca092f3209579fde7a38ade511c1bbcef213c36
      https://github.com/llvm/llvm-project/commit/2ca092f3209579fde7a38ade511c1bbcef213c36
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-01-17 (Fri, 17 Jan 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
    M llvm/test/Transforms/InstCombine/expensive-combines.ll

  Log Message:
  -----------
  [InstCombine] Support disabling expensive combines in opt

Currently, there is no way to disable ExpensiveCombines when doing
a standalone opt -instcombine run, as that's the default, and the
opt option can currently only be used to force enable, not to force
disable. The only way to disable expensive combines is via -O1 or -O2,
but that of course also runs the rest of the kitchen sink...

This patch allows using opt -instcombine -expensive-combines=0 to
run InstCombine without ExpensiveCombines.

Differential Revision: https://reviews.llvm.org/D72861


Compare: https://github.com/llvm/llvm-project/compare/8bcf976841f5...2ca092f32095