[PATCH] D79089: [flang] Add the proposal document and rationale for the internal naming module that was previously added.

[Tim Keith via Phabricator via llvm-commits]

tskeith added inline comments.


================
Comment at: flang/documentation/BijectiveInternalNameUniquing.md:34
+```
+    submodule (mod:s1mod) s2mod
+      ...
----------------
Submodules of an ancestor module have to have distinct names, so you don't need to include `s1mod` in the unique name for `s2mod` (though it doesn't hurt). So the unique name for `smod2` could be just `_QMmodSs2mod`.


================
Comment at: flang/documentation/BijectiveInternalNameUniquing.md:49
+
+   * A common block name will be prefixed with `B`
+
----------------
What about the blank common block? Is its name just `_QB`?


================
Comment at: flang/documentation/BijectiveInternalNameUniquing.md:58
+
+   * A procedure/subprogram is prefixed with `P`
+
----------------
On line 30 it says `F` is the prefix. `P` makes more sense to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79089