[PATCH] D54499: [codeview] Make "clang -g" emit codeview by default when targetting MSVC

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai accepted this revision.
smeenai added a comment.
This revision is now accepted and ready to land.

LGTM, though you may wanna also wait for Zach.



================
Comment at: clang/test/CodeGen/dwarf-version.c:33
+// RUN: %clang -target i686-pc-windows-msvc -gdwarf -gcodeview -S -emit-llvm -o - %s \
+// RUN:     | FileCheck %s --check-prefixes=VER4,CODEVIEW
 int main (void) {
----------------
Is it worth adding a test for a non msvc target to ensure the default for `-g` is still DWARF?


https://reviews.llvm.org/D54499