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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
