mtrofin added inline comments.

================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:1111
+      !getModule().getSourceFileName().empty()) {
+    llvm::MD5 Md5;
+    Md5.update(getModule().getSourceFileName());
----------------
Just a thought: md5 is a non-bijective transformation, afaik. How about using 
base64 encoding, with the caveat that we replace + with $ and / with _ (so it 
results in a valid name), and discard padding =

The value being, one can copy/paste that identifier, do the trivial conversion 
back to base64 ($->+, _->/) and get the module name. Useful when debugging, for 
example.


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

https://reviews.llvm.org/D73307



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

Reply via email to