[PATCH] D100332: [CSSPGO] Flip SkipPseudoOp to true for MIR APIs.

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 18 13:11:58 PDT 2021


wmi added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/MachineBasicBlock.h:685-686
   /// unlikely hurt profile quality, e.g., without block merging.
   /// TODO: flip the default value of \c SkipPseudoOp to maximize code quality
   /// with pseudo probes.
+  iterator getFirstNonDebugInstr(bool SkipPseudoOp = true);
----------------
TODO can be removed now. It maybe better to list the optimizations like branch folding and if-conversion here which set SkipPseudoOp to false as a reference.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100332



More information about the llvm-commits mailing list