[PATCH] D17183: Make TargetInfo store an actual DataLayout instead of a string.

James Y Knight via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 11 20:10:36 PST 2016


jyknight added a comment.

In http://reviews.llvm.org/D17183#351065, @yaron.keren wrote:

> We have tried to keep one copy of DataLayout around
>  http://reviews.llvm.org/D11103
>  Can it share the Module->getDataLayout() ?


We're at the beginning of clang here, before preprocessing. There's no llvm::Module yet, one of those doesn't show up until way later, when the AST is ready, and is going to be turned into IR.


http://reviews.llvm.org/D17183





More information about the cfe-commits mailing list