[PATCH] D119399: [MachineSink] Allow target to adjust sink insertion point

Ruiling, Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 10 19:36:14 PST 2022


ruiling added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineSink.cpp:1397
-  MachineBasicBlock::iterator InsertPos = SuccToSinkTo->begin();
-  while (InsertPos != SuccToSinkTo->end() && InsertPos->isPHI())
-    ++InsertPos;
----------------
I would suggest we use SkipPHIsAndLabels() to determine the insertion point here. It will make sure we are inserting after prologue instructions. I would also suggest we fix the other two getFirstNonPHI() uses in this file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119399



More information about the llvm-commits mailing list