On 10/07/2015 07:37 PM, Bernd Schmidt wrote:
On 10/07/2015 12:45 PM, Christian Bruel wrote:
On 10/07/2015 12:18 PM, Bernd Schmidt wrote:
On 10/07/2015 09:04 AM, Christian Bruel wrote:
+ /* Similarly, relayout function's alignment if not forced. */
+ if (!DECL_USER_ALIGN (fndecl)
+ && (TREE_CODE (fntype) != METHOD_TYPE
+ || TARGET_PTRMEMFUNC_VBIT_LOCATION !=
ptrmemfunc_vbit_in_pfn))
+ DECL_ALIGN (fndecl) = FUNCTION_BOUNDARY;
}
Hmm, more questions are coming to mind - are other places that set
DECL_ALIGN for functions redundant after this change? It looks like the
main place where it's set is make_node_stat where it's also set to
FUNCTION_BOUNDARY. That's small enough not to bother changing it, but it
raises another issue: we don't allocate a struct function for mere
declarations, so does that mean they can have an incorrect DECL_ALIGN?
Bernd