[llvm] Refine the examples in the debug info document (PR #86272)
Stephen Tozer via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 22 07:19:46 PDT 2024
================
@@ -91,8 +91,11 @@ misattributed to a block containing one of the instructions-to-be-merged.
Examples of transformations that should follow this rule include:
-* Merging identical loads/stores which occur on both sides of a CFG diamond
- (see the ``MergedLoadStoreMotion`` pass).
+* Hoisting identical instructions from successors of a conditional branch. For
+ example, merging identical loads/stores which occur on both sides of a CFG
+ diamond (see the ``MergedLoadStoreMotion`` pass). If there are more than one
+ group of identical instructions hoisted, apply merging instruction locations
+ for each single merged instruction.
----------------
SLTozer wrote:
I think "merging" should be replaced with "merged". Alternatively as an optional suggestion for the last sentence, "For each group of identical instructions being hoisted/sunk, the merge of all their locations should be applied to the merged instruction."
https://github.com/llvm/llvm-project/pull/86272
More information about the llvm-commits
mailing list