diff options
author | Jan Hubicka <jh@suse.cz> | 2012-10-05 13:43:38 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2012-10-05 13:43:38 +0200 |
commit | bfc61b40d43b722b8a57868ae2df92c345684ec9 (patch) | |
tree | 997232f16181a9510cc7119f7063bd84d3333404 /gcc/tree-inline.c | |
parent | 9ed313cc23232a8e8f16a5b1e94fc93b1a38e0f7 (diff) | |
download | gcc-bfc61b40d43b722b8a57868ae2df92c345684ec9.zip gcc-bfc61b40d43b722b8a57868ae2df92c345684ec9.tar.gz gcc-bfc61b40d43b722b8a57868ae2df92c345684ec9.tar.bz2 |
re PR c/33763 (Bogus inlining failed in call to `xxx': redefined extern inline functions are not considered for inlining)
PR tree-optimization/33763
* tree-inline.c (expand_call_inline): Silently ignore always_inline
attribute for redefined extern inline functions.
* c-c++-common/pr33763.c: New test.
From-SVN: r192119
Diffstat (limited to 'gcc/tree-inline.c')
-rw-r--r-- | gcc/tree-inline.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/gcc/tree-inline.c b/gcc/tree-inline.c index a585c0b..d6fbf50 100644 --- a/gcc/tree-inline.c +++ b/gcc/tree-inline.c @@ -3814,6 +3814,12 @@ expand_call_inline (basic_block bb, gimple stmt, copy_body_data *id) goto egress; if (lookup_attribute ("always_inline", DECL_ATTRIBUTES (fn)) + /* For extern inline functions that get redefined we always + silently ignored always_inline flag. Better behaviour would + be to be able to keep both bodies and use extern inline body + for inlining, but we can't do that because frontends overwrite + the body. */ + && !cg_edge->callee->local.redefined_extern_inline /* Avoid warnings during early inline pass. */ && cgraph_global_info_ready /* PR 20090218-1_0.c. Body can be provided by another module. */ |