aaronpuchert added inline comments.

================
Comment at: flang/lib/Frontend/FrontendActions.cpp:590
+  std::optional<llvm::CodeGenOpt::Level> OptLevelOrNone =
+      CodeGenOpt::getLevel(CGOpts.OptimizationLevel);
+  assert(OptLevelOrNone && "Invalid optimization level!");
----------------
Does this need `llvm::`?

```
../llvm-project/flang/lib/Frontend/FrontendActions.cpp:590:7: error: use of 
undeclared identifier 'CodeGenOpt'; did you mean 'llvm::CodeGenOpt'?
      CodeGenOpt::getLevel(CGOpts.OptimizationLevel);
      ^~~~~~~~~~
      llvm::CodeGenOpt
../llvm-project/llvm/include/llvm/Support/CodeGen.h:53:13: note: 
'llvm::CodeGenOpt' declared here
  namespace CodeGenOpt {
            ^
```
See https://lab.llvm.org/buildbot/#/builders/191/builds/13832.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141968

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to