[PATCH] D74918: Add method to TargetInfo to get CPU cache line size
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 19 11:27:22 PDT 2020
lebedev.ri added a comment.
In D74918#1931488 <https://reviews.llvm.org/D74918#1931488>, @zoecarver wrote:
> @lebedev.ri LLVM may be better, I'm not sure. If you feel strongly I can move it.
I do think that it makes much more sense somewhere closer to the backend
> @jyknight I'm planning on adding a builtin that uses this method. I can put the patch up for that first if you would like.
but that being said, currently nothing needs that and with all those concerns about exposing this info,
i'd hold off moving it until it is settled whether it is needed (and not a dead code) in the first place :/
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