[PATCH] D147954: [SimplifyCFG]Prevent premature simplification of indirect callsites with profile data

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 18 09:48:02 PDT 2023


davidxl added a comment.

In D147954#4276844 <https://reviews.llvm.org/D147954#4276844>, @nikic wrote:

> Could you please start by implementing metadata merging, and then return to this patch only if strictly necessary? I'd rather not have another SimplifyCFG option for such a niche use case, if it's possible to get most of the benefit via standard metadata merging.

Merging value profile data will likely render the data not quite useful anymore -- the same effect of dropping the data completely.


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

https://reviews.llvm.org/D147954



More information about the llvm-commits mailing list