<div dir="auto">Oh! Huh, missed that. Probably my ninja check-all didn’t build the benchmarks. I’ll patch shortly. Sorry for the lapse.</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 16, 2021 at 14:11 Siva Chandra via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">sivachandra added a comment.<br>
<br>
FYI: The libcxx copy and llvm/utils copy were not identical. So, the libc benchmarks are currently broken because of that. I have added an inline comment to show the single difference we are seeing. We can fix forward.<br>
<br>
<br>
<br>
================<br>
Comment at: llvm/utils/benchmark/include/benchmark/benchmark.h:1236<br>
-<br>
-struct CPUInfo {<br>
-  struct CacheInfo {<br>
----------------<br>
The definition of this struct is not the same as the one in the libcxx copy.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D112012/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D112012/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D112012" rel="noreferrer" target="_blank">https://reviews.llvm.org/D112012</a><br>
<br>
</blockquote></div></div>