[PATCH] D74918: Add method to TargetInfo to get CPU cache line size
Eli Friedman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 20 13:07:45 PST 2020
efriedma added a comment.
Where are you planning to use this?
================
Comment at: clang/lib/Sema/SemaStmt.cpp:2817
+
+ unsigned targetCacheLineSize = Ctx.getTargetInfo().getCPUCacheLineSize();
+ if (!targetCacheLineSize)
----------------
"64" here is arbitrary; it's not actually supposed to be the cache line size for any particular CPU, just something generally expensive. I don't think this warning should depend on -mcpu.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74918/new/
https://reviews.llvm.org/D74918
More information about the cfe-commits
mailing list