zoecarver added a comment. > BTW, I note that facebook uses 128 bytes for x86
They also use 64 for arm which is interesting (the opposite of this patch). Also, see this comment in the same snippet: > We assume a cache line size of 64, so we use a cache line pair size of 128 Which would indicate that 64 is the correct number of bytes for x86. But maybe we should double that for `hardware_destructive_interference_size`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66822/new/ https://reviews.llvm.org/D66822 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
