[all-commits] [llvm/llvm-project] 9c56e9: [NPM] Bail out when -foo and --passes=foo are both...

[aeubanks via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 9c56e94a9f0d9a0930179e764aff7c80c793e117
      https://github.com/llvm/llvm-project/commit/9c56e94a9f0d9a0930179e764aff7c80c793e117
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2020-06-22 (Mon, 22 Jun 2020)

  Changed paths:
    A llvm/test/Other/opt-old-new-pm-passes.ll
    M llvm/test/Transforms/IRCE/bug-loop-varying-upper-limit.ll
    M llvm/test/Transforms/IRCE/non-loop-invariant-rhs-instr.ll
    M llvm/test/Transforms/LoadStoreVectorizer/X86/codegenprepare-produced-address-math.ll
    M llvm/test/Transforms/RewriteStatepointsForGC/relocate-invoke-result.ll
    M llvm/test/Transforms/RewriteStatepointsForGC/rewrite-invoke.ll
    M llvm/tools/opt/opt.cpp

  Log Message:
  -----------
  [NPM] Bail out when -foo and --passes=foo are both specified

Summary:
Currently when --passes is used, any passes specified via -foo are
ignored. Explicitly bail out when that happens.

This requires changing some tests. Most were straightforward, but
codegenprepare-produced-address-math.ll is tricky. One of its RUNs runs
CodeGenPrepare. I tried porting CodeGenPrepare to the NPM, but ended up
getting stuck when I needed a TargetMachine. NPM doesn't have support
for MachineFunctions yet. So I just deleted that RUN line, since it was
mass-added in https://reviews.llvm.org/D54848 and is likely not that
useful.

Reviewers: echristo, hans

Subscribers: llvm-commits

Tags: #llvm

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