From 59063140a7a61f13ad9344b42b42372c8ffbca9a Mon Sep 17 00:00:00 2001 From: Eric Christopher Date: Mon, 26 Jan 2015 19:19:04 +0000 Subject: [PATCH] Add a FIXME about preferred alignment to DataLayout. Essentially DataLayout is global and affects the layout of ABI level objects. Preferred alignment could change on a per function basis as we change CPU features. llvm-svn: 227118 --- llvm/include/llvm/IR/DataLayout.h | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/llvm/include/llvm/IR/DataLayout.h b/llvm/include/llvm/IR/DataLayout.h index a9e7595..9479ba4 100644 --- a/llvm/include/llvm/IR/DataLayout.h +++ b/llvm/include/llvm/IR/DataLayout.h @@ -53,6 +53,11 @@ enum AlignTypeEnum { AGGREGATE_ALIGN = 'a' }; +// FIXME: Currently the DataLayout string carries a "preferred alignment" +// for types. As the DataLayout is module/global, this should likely be +// sunk down to an FTTI element that is queried rather than a global +// preference. + /// \brief Layout alignment element. /// /// Stores the alignment data associated with a given alignment type (integer, -- 2.7.4