[PATCH] D112036: [opt] Directly translate -O# to -passes='default<O#>'

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/test/CodeGen/NVPTX/nvvm-reflect-arch.ll:4
 ;
-; RUN: opt %s -S -nvvm-reflect -O2 -mtriple=nvptx64 \
 ; RUN:   | FileCheck %s --check-prefixes=COMMON,SM20
----------------
we run this as part of the -O2 pipeline due to -mtriple, no need to specify it again
also, it's very confusing because it actually runs after -O2, which took me a while to figure out why the test was failing with `-passes=function(nvvm-reflect),default<O2>`, since we actually need some passes to run before nvvm-reflect for some reason


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112036/new/

https://reviews.llvm.org/D112036