[PATCH] D96865: [Driver] Honor "-gdwarf-N" at any position for assembler sources

[Igor Kudrin via Phabricator via cfe-commits]

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