[PATCH] D133310: [Assignment Tracking][15/*] Account for assignment tracking in simplifycfg

Chris Jackson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 15 06:31:05 PDT 2022


chrisjackson added a comment.

Just a couple of typos.



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:1899
+    // In most (all?) cases this just introduces a use-before-def.
+    assert(I->user_empty() && "I unexpectedly still has non-dbg users");
+    I->replaceAllUsesWith(I0);
----------------



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:2576
   // be misleading while debugging.
   // Similarly strip attributes that maybe dependent on condition we are
   // hoisting above.
----------------



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

https://reviews.llvm.org/D133310



More information about the llvm-commits mailing list