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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits