ikudrin added a comment. Thanks!
================ Comment at: clang/test/Driver/debug-options-as.c:65 +// Check that -gdwarf-N can be placed before other options of the "-g" group. +// RUN: %clang -### -c -g -gdwarf-3 -integrated-as -x assembler %s 2>&1 \ +// RUN: | FileCheck -check-prefix=DWARF3 %s ---------------- MaskRay wrote: > Nit: use the more common spelling `-fintegrated-as` > > `-triple %itanium_abi_triple` > > msvc windows triples ideally don't need `-dwarf-version=`, but that can be a > separate clean-up. > Nit: use the more common spelling -fintegrated-as `-integrated-as` is what other tests in this file use. Is it OK if these lines will not follow the common template? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96865/new/ https://reviews.llvm.org/D96865 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits