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

Xinliang David Li via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 29 15:09:50 PDT 2020


right. It occurred to me during review, but did not think of the hard coded
values in proftext depends on LE.

On Wed, Jul 29, 2020 at 3:04 PM Hiroshi Yamauchi via Phabricator <
reviews at reviews.llvm.org> wrote:

> yamauchi added a comment.
>
> Builtbot failure:
> http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/51984
>
> I think it's an endian issue in the hash computation.
>
> Will revert.
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D84782/new/
>
> https://reviews.llvm.org/D84782
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200729/96656ee2/attachment.html>


More information about the cfe-commits mailing list