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

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 30 10:14:06 PDT 2020


MaskRay added a comment.

It is also a convention to explain what has changed compared with the initial land.

As a hindsight, the option `-pgo-instr-old-cfg-hashing` should probably have been mentioned in the description.


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