[PATCH] D39446: [PGO] Detect more structural changes with the stable hash

Vedant Kumar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 1 18:16:30 PDT 2017


vsk updated this revision to Diff 121224.
vsk marked 2 inline comments as done.
vsk edited the summary of this revision.
vsk added a comment.

- Handle loop nesting, conditions, and out-of-order control flow.
- Improve test coverage. Add a format compatibility test, and check that functions which were previously hashed the same way get different hashes now.


https://reviews.llvm.org/D39446

Files:
  lib/CodeGen/CodeGenPGO.cpp
  test/Profile/Inputs/c-captured.proftext
  test/Profile/Inputs/c-counter-overflows.proftext
  test/Profile/Inputs/c-general.proftext
  test/Profile/Inputs/c-unprofiled-blocks.proftext
  test/Profile/Inputs/cxx-class.proftext
  test/Profile/Inputs/cxx-lambda.proftext
  test/Profile/Inputs/cxx-rangefor.proftext
  test/Profile/Inputs/cxx-templates.proftext
  test/Profile/Inputs/cxx-throws.proftext
  test/Profile/Inputs/func-entry.proftext
  test/Profile/Inputs/gcc-flag-compatibility.proftext
  test/Profile/Inputs/objc-general.proftext
  test/Profile/Inputs/objcxx-hash-v2.profdata.v5
  test/Profile/Inputs/objcxx-hash-v2.proftext
  test/Profile/c-outdated-data.c
  test/Profile/objc-general.m
  test/Profile/objcxx-hash-v2.mm

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39446.121224.patch
Type: text/x-patch
Size: 20134 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171102/b57e1706/attachment-0001.bin>


More information about the cfe-commits mailing list