[PATCH] D124439: [JumpThreading][NFC][CompileTime] Do not recompute BPI/BFI analyzes

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 29 14:14:46 PDT 2022


jyknight added a comment.

This change has caused our internal PGO+NewPM builds of clang to have become non-reproducible (that is: the binary is different every build, when built with a Clang that includes this change). The problem appears only when providing profile data. I am not sure //why// this change causes a problem -- perhaps some of the analysis data ought to have been invalidated by some other pass, but wasn't?

But, I've bisected to exactly this commit, so, I'm going to revert it while continuing to investigate.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124439



More information about the llvm-commits mailing list