[PATCH] D134544: [clang-cl] Implement /ZH: flag

[David Blaikie via Phabricator via cfe-commits]

dblaikie added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:3220
       Values<"simple,mangled">, Flags<[CC1Option, NoDriverOption]>;
+def gsrc_hash_EQ : Joined<["-"], "gsrc-hash=">,
+  Group<g_flags_Group>, Flags<[CC1Option, NoDriverOption]>,
----------------
This is a driver flag, yeah? Might be worth documenting/emitting a warning that it's not compatible with DWARF emission? (looks like the DWARF code in LLVM ignores any hash that's not MD5, because DWARF doesn't have a way to encode other hash algorithms currently - though maybe there's some extension space available to add it, I haven't checked/looked)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134544/new/

https://reviews.llvm.org/D134544