================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to