[PATCH] D37052: Add default address space for functions to the data layout (1/4)

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 24 10:00:00 PDT 2017


arsenm added a comment.

Is the intent to allow different functions to have different address spaces and this is the default, or all functions have the one address space in the datalayout? I think the description should clarify.



================
Comment at: lib/Target/AVR/AVRTargetMachine.cpp:28
 
-static const char *AVRDataLayout = "e-p:16:16:16-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-n8";
 
----------------
I think changing the AVR layout is a separate patch


https://reviews.llvm.org/D37052





More information about the llvm-commits mailing list