From: Sriraman Tallam <tmsriram@google.com>
To: Diego Novillo <dnovillo@google.com>
Cc: Xinliang David Li <davidxl@google.com>,
Richard Guenther <richard.guenther@gmail.com>,
reply@codereview.appspotmail.com, gcc-patches@gcc.gnu.org
Subject: Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)
Date: Wed, 04 May 2011 22:19:00 -0000 [thread overview]
Message-ID: <BANLkTinzVT=em4z00p-Mqot6H1RDLEQeqg@mail.gmail.com> (raw)
In-Reply-To: <BANLkTikRyYOKwKb7vSxd=jUPkjQUibL3ow@mail.gmail.com>
I submitted the patch.
Thanks,
-Sri.
On Wed, May 4, 2011 at 3:13 PM, Diego Novillo <dnovillo@google.com> wrote:
> On Wed, May 4, 2011 at 15:35, Sriraman Tallam <tmsriram@google.com> wrote:
>
>> * tree-pass.h (pass_tree_convert_builtin_dispatch): New pass.
>> (pass_ipa_multiversion_dispatch): New pass.
>> * builtin-types.def (BT_PTR_FN_INT): New pointer type.
>> (BT_FN_INT_PTR_FN_INT_PTR_PTR_VAR): New function type for __builtin_dispatch.
>> * builtins.def (BUILT_IN_DISPATCH): New builtin to
>> support multi-version calls.
>> * mversn-dispatch.c: New file.
>> * timevar.de (TV_MVERSN_DISPATCH): New time var.
>> * common.opt (fclone-hot-version-paths): New flag.
>> * Makefile.in (mversn-dispatch.o): New rule.
>> * passes.c (init_optimization_passes): Add the new
>> multi-version and dispatch passes to the pass list.
>> * params.def (PARAM_NUMBER_OF_MVERSN_CLONES): Define.
>> (PARAM_MVERSN_CLONE_CGRAPH_DEPTH): Define.
>> * doc/invoke.texi (mversn-clone-depth): Document.
>> (num-mversn-clones): Document.
>> (fclone-hot-version-paths): Document.
>> * testsuite/gcc.dg/mversn7.c: New test.
>> * testsuite/gcc.dg/mversn4.c: New test.
>> * testsuite/gcc.dg/mversn4.h: New test.
>> * testsuite/gcc.dg/mversn4a.c: New test.
>> * testsuite/gcc.dg/torture/mversn1.c: New test.
>> * testsuite/gcc.dg/mversn2.c: New test.
>> * testsuite/gcc.dg/mversn6.c: New test.
>> * testsuite/gcc.dg/mversn3.c: New test.
>> * testsuite/g++.dg/mversn8.C: New test.
>> * testsuite/g++.dg/mversn10a.C: New test.
>> * testsuite/g++.dg/mversn14a.C: New test.
>> * testsuite/g++.dg/tree-prof/mversn13.C: New test.
>> * testsuite/g++.dg/tree-prof/mversn15.C: New test.
>> * testsuite/g++.dg/tree-prof/mversn15a.C: New test.
>> * testsuite/g++.dg/mversn9.C: New test.
>> * testsuite/g++.dg/mversn10.C: New test.
>> * testsuite/g++.dg/mversn12.C: New test.
>> * testsuite/g++.dg/mversn14.C: New test.
>> * testsuite/g++.dg/mversn16.C: New test.
>> * testsuite/g++.dg/torture/mversn11.C: New test.
>> * testsuite/g++.dg/torture/mversn5.C: New test.
>> * testsuite/g++.dg/torture/mversn5.h: New test.
>> * testsuite/g++.dg/torture/mversn5a.C: New test.
>> * c-family/c-common.c (handle_version_selector_attribute): New function.
>> (c_common_attribute_table): New attribute "version_selector".
>
> OK. Thanks for the quick fix!
>
>
> Diego.
>
next prev parent reply other threads:[~2011-05-04 22:16 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-29 5:03 Sriraman Tallam
2011-04-29 11:59 ` Richard Guenther
2011-04-29 16:55 ` Xinliang David Li
2011-05-02 9:11 ` Richard Guenther
2011-05-02 16:42 ` Xinliang David Li
2011-05-02 19:12 ` Sriraman Tallam
2011-05-02 19:32 ` Xinliang David Li
2011-05-02 20:38 ` Sriraman Tallam
2011-05-02 20:45 ` Eric Botcazou
2011-05-02 20:51 ` Sriraman Tallam
2011-05-04 19:50 ` Sriraman Tallam
2011-05-04 22:16 ` Diego Novillo
2011-05-04 22:19 ` Sriraman Tallam [this message]
2011-05-02 21:34 ` Richard Guenther
2011-05-02 23:08 ` Xinliang David Li
2011-05-03 10:00 ` Richard Guenther
2011-05-03 10:07 ` Richard Guenther
2011-05-03 15:21 ` Mike Stump
2011-05-03 15:34 ` Joseph S. Myers
2011-05-03 16:50 ` Mike Stump
2011-05-03 22:05 ` Xinliang David Li
2011-05-04 9:30 ` Richard Guenther
2011-05-04 22:20 ` Xinliang David Li
2011-05-05 9:21 ` Richard Guenther
2011-05-05 17:08 ` Xinliang David Li
2011-05-06 9:07 ` Richard Guenther
2011-05-06 13:38 ` Diego Novillo
2011-05-06 18:01 ` Xinliang David Li
2011-05-07 12:48 ` Richard Guenther
2011-05-07 18:00 ` Xinliang David Li
2011-05-07 18:11 ` Richard Guenther
2011-04-29 18:46 ` Sriraman Tallam
[not found] ` <BANLkTim=oXD7egdy7zzMcTMDAhwnMQ=Kyg@mail.gmail.com>
2011-05-02 9:25 ` Richard Guenther
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='BANLkTinzVT=em4z00p-Mqot6H1RDLEQeqg@mail.gmail.com' \
--to=tmsriram@google.com \
--cc=davidxl@google.com \
--cc=dnovillo@google.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=reply@codereview.appspotmail.com \
--cc=richard.guenther@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).