[PATCH] D84782: [PGO] Include the mem ops into the function hash.

David Li via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 29 13:33:35 PDT 2020


davidxl accepted this revision.
davidxl added a comment.

lgtm

Just realized that we need a test case to show it fixes the original issue (existence with memop --> different hash). Ok as a follow up .


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84782



More information about the cfe-commits mailing list