[PATCH] D79902: [IR] Make Module::setProfileSummary to replace an existing ProfileSummary flag.

Rong Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 19 12:37:10 PDT 2020


xur added a comment.

I noticed the same issue when I implemented CSFDO -- I initially reused the same summary name for CSFDO as in FDO.
Exiting of two module flags with the same name sometime breaks the build (not always, if I remember correctly).

This fix looks to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79902





More information about the llvm-commits mailing list