[PATCH] D79772: Nondeterminism of iterators causes false ThinLTO cache misses

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 29 10:53:51 PDT 2020


mehdi_amini accepted this revision.
mehdi_amini added a comment.

LGTM

In D79772#2062317 <https://reviews.llvm.org/D79772#2062317>, @kromanova wrote:

> Instead of storing pointers in the vector as you suggested, I think it will be better to store iterators.


Either way is fine with me


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

https://reviews.llvm.org/D79772





More information about the llvm-commits mailing list