This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [google] Add options to pattern match function name for hotness attributes
- From: Andi Kleen <andi at firstfloor dot org>
- To: Dehao Chen <dehao at google dot com>
- Cc: gcc-patches at gcc dot gnu dot org, David Li <davidxl at google dot com>
- Date: Sun, 03 Jun 2012 18:34:28 -0700
- Subject: Re: [google] Add options to pattern match function name for hotness attributes
- References: <CAO2gOZUrnW+V-wUbZ5vLCN5yne+uivUtpptzemkrRU5o5UhFLg@mail.gmail.com>
Dehao Chen <dehao@google.com> writes:
> Hi,
>
> This patch adds 4 flags to enable user to type in a list of name
> patterns. Compiler will match the function name with the given
> patterns, and add "hot", "cold", "likely_hot", "likely_cold"
> attributes to function declaration. The static branch prediction
> checks if a basic block contains call to a annotated function, and set
> the branch probability accordingly.
I like the idea (and would have some uses for it), but I don't like the
command line options. That will lead to longer and longer command
lines. Could this be made a pragma instead?
You could still specify it from the Makefile by using -include
-Andi
--
ak@linux.intel.com -- Speaking for myself only