[PATCH] D74918: Add method to TargetInfo to get CPU cache line size

Sat Mar 21 00:29:43 PDT 2020

craig.topper added inline comments.

================
Comment at: clang/lib/Basic/Targets/X86.cpp:1786
+    // i386
+    case CK_i386:
+    // Netburst
----------------
zoecarver wrote:
> craig.topper wrote:
> > I found the documentation for the 82385 cache controller chip for the 386. It's a bit weird. The tags for the cache are based on 16 byte lines, but there are valid bits for every 2 bytes within that line. So its possible for only a portion of a line to be valid.
> Interesting. Does this mean that we should be returning a different value?
I think we should make 386 return 16.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74918