rovka added a comment. Nit: Should we also have a test for print-effective-triple?
Otherwise LGTM, although I'm not sure that -emit-llvm is necessarily something we'd want flang users to be exposed to. ================ Comment at: flang/include/flang/Frontend/TargetOptions.h:20 +/// Options for controlling the target. Currently this is just a placeholder. +/// In the future we will use this for options to specify various target +/// options that will affect the generated code e.g.: ---------------- Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120246/new/ https://reviews.llvm.org/D120246 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits