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

[Xiangling Liao via Phabricator via llvm-commits]

Xiangling_L marked an inline comment as done.
Xiangling_L 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
+
----------------
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`]?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85527