[PATCH] D45835: Add new driver mode for dumping compiler options

[Hal Finkel via Phabricator via cfe-commits]

hfinkel added inline comments.


================
Comment at: test/Frontend/compiler-options-dump.cpp:3
+// RUN: %clang_cc1 -compiler-options-dump -std=c++17 %s -o - | FileCheck %s --check-prefix=CXX17
+// RUN: %clang_cc1 -compiler-options-dump -std=c99 -ffast-math -x c %s -o - | FileCheck %s --check-prefix=C99
+
----------------
You don't need -ffast-math here I presume.


================
Comment at: test/Frontend/compiler-options-dump.cpp:15
+// CXX17: "extensions"
+// CXX17: "cxx_range_for" : true
+
----------------
cxx_range_for is both a feature and an extension?


https://reviews.llvm.org/D45835