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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to