[PATCH] D128043: [flang][driver] Add support for `-O{0|1|2|3}`

[Diana Picus via Phabricator via cfe-commits]

rovka added a comment.

I just realized I haven't pestered you enough about testing :) Can you add a test that -O4 indeed warns and uses -O3? Also, the summary says this should work in both the compilation and the frontend driver, but you're only testing with %flang_fc1.



================
Comment at: flang/test/Driver/default-optimization-pipelines.f90:1
+! Verify that`-O{n}` is indeed taken into account when defining the LLVM optimization/middle-end pass pass pipeline.
+
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128043