[llvm-branch-commits] [llvm] [NFC][KeyInstr] Add Atom Group (re)mapping (PR #133479)

[Jeremy Morse via llvm-branch-commits]

================
@@ -117,9 +118,21 @@ struct ClonedCodeInfo {
 /// If you would like to collect additional information about the cloned
 /// function, you can specify a ClonedCodeInfo object with the optional fifth
 /// parameter.
+///
+/// Set \p MapAtoms to false to skip mapping source atoms for later remapping.
----------------
jmorse wrote:

IMO "source-location atoms" to make it even clearer that this is a debugging feature.

Also IMO it's better to discuss when this flag is necessary instead of when it's not necessary, as it'll enlighten the reader what it's for. AFAIUI, something like "Must be true when you duplicate a code path and a source line is intended to appear twice in the generated instructions. Can be set to false if you are transplanting code from one place to another".

https://github.com/llvm/llvm-project/pull/133479