On 07/24/2017 08:56 PM, Jan Hubicka wrote:
* gcc.dg/cold-1.c: New testcase.
Index: testsuite/gcc.dg/cold-1.c
===
--- testsuite/gcc.dg/cold-1.c (revision 0)
+++ testsuite/gcc.dg/cold-1.c (working copy)
@@ -0,0 +1,21 @@
+extern void do_something_interesting_and_never_return ();
+
+int
+foo1(int a)
+{ /* { dg-warning "cold" "detect cold candidate" { target *-*-* } "8" } */
+ if (a)
+abort ();
+ else
+abort ();
+}
In this case it looks to me like with the patch GCC will actually
issue two suggestions: t
On 07/24/2017 04:06 PM, Jan Hubicka wrote:
>> On 07/24/2017 01:08 PM, Jan Hubicka wrote:
On Mon, Jul 24, 2017 at 2:56 PM, Jan Hubicka wrote:
> Hi,
> this patch adds -Wsuggest-attribute=cold because we can now statically
> detect
> cold functions atuomatically.
>
> Boo
> On 07/24/2017 01:08 PM, Jan Hubicka wrote:
> >> On Mon, Jul 24, 2017 at 2:56 PM, Jan Hubicka wrote:
> >>> Hi,
> >>> this patch adds -Wsuggest-attribute=cold because we can now statically
> >>> detect
> >>> cold functions atuomatically.
> >>>
> >>> Bootstrapped/regtested x86_64-linux. Plan to co
On 07/24/2017 01:08 PM, Jan Hubicka wrote:
>> On Mon, Jul 24, 2017 at 2:56 PM, Jan Hubicka wrote:
>>> Hi,
>>> this patch adds -Wsuggest-attribute=cold because we can now statically
>>> detect
>>> cold functions atuomatically.
>>>
>>> Bootstrapped/regtested x86_64-linux. Plan to commit it tomorrow
> On Mon, Jul 24, 2017 at 2:56 PM, Jan Hubicka wrote:
> > Hi,
> > this patch adds -Wsuggest-attribute=cold because we can now statically
> > detect
> > cold functions atuomatically.
> >
> > Bootstrapped/regtested x86_64-linux. Plan to commit it tomorrow if there
> > are no
> > complains.
> >
> >
On Mon, Jul 24, 2017 at 2:56 PM, Jan Hubicka wrote:
> Hi,
> this patch adds -Wsuggest-attribute=cold because we can now statically detect
> cold functions atuomatically.
>
> Bootstrapped/regtested x86_64-linux. Plan to commit it tomorrow if there are
> no
> complains.
>
> Honza
>
> * invo
