[PATCH] D158442: [BasicBlockSections] Introduce the path cloning profile format to BasicBlockSectionsProfileReader.

Sriraman Tallam via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 23 12:51:09 PDT 2023


tmsriram added a comment.

The description requires more details.  Please point to any prior discussions on cloning and its performance potential. Can you explain in some more detail how this syntax is powerful enough to capture a general cloning operation?

Rather than just use "number of !s", can we try to change the grammar?  How about a character to specify the action type, 'c' for cloning?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158442



More information about the llvm-commits mailing list