[PATCH] D85527: [AIX] Generate unique module id based on PID and timestamp

[Hubert Tong via Phabricator via llvm-commits]

hubert.reinterpretcast added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-static-init-no-unique-module-id.ll:15
+; way is not reproducible. A function name sample would be:
+; __sinit80000000_clang_119189_1597348415_0
+
----------------
Xiangling_L wrote:
> hubert.reinterpretcast wrote:
> > Sorry for not mentioning this earlier, but could we replace `clang` here with `clangPidTime` or similar? More generally, I think we should want to indicate the format style in that position.
> Do we only add indicator of format style when `getUniqueModuleId`[or source file full path later] does not work? i.e. do we also want to add one when `getUniqueModuleId`[or source file full path] does work and replace `clang` with something like `clangUniqueModId`[or `clangSrcFullPath`]?
I think we would also want a format indicator for the other cases. This would help in case we find reasons for having more than one format available.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85527