This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Update source location for PRE inserted stmt
- From: Dehao Chen <dehao at google dot com>
- To: Richard Biener <richard dot guenther at gmail dot com>
- Cc: Xinliang David Li <davidxl at google dot com>, Steven Bosscher <stevenb dot gcc at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 4 Nov 2012 09:42:47 -0800
- Subject: Re: [PATCH] Update source location for PRE inserted stmt
- References: <CAO2gOZWF4=vpgWhX+duS7B5z3dN_cvy7AjkefZ18hi2QsY6SPA@mail.gmail.com> <CABu31nOZrqBVOiVPeCb-E0k0ou=JOqhFk-F0HTNDjtjqRvTGUw@mail.gmail.com> <CAAkRFZKVd-vUCJpMaRhn1UKq+Ceuhga2U79+D3OUCkFGHBPDcA@mail.gmail.com> <CAFiYyc2y6CvcLo5q6UFkS9CSz1TMHm+SDcDqo7EFQdiA5XipEQ@mail.gmail.com> <CAAkRFZK17Kp70NzaddJrN8MDKJSzt9LUwsGUep5CjKRJKsS8AQ@mail.gmail.com> <CAFiYyc0aWA5UvUvH9qScZ3wU_wD0-pcoa2YXZm=-O4ou3Gi0aQ@mail.gmail.com>
2. __attribute__((noinline)) int abc (int *a)
3. {
4. int ret = 0;
5.
6. if (x > 0)
7. ret += *a;
8. else
9. a++;
10.
11. ret += *a;
12. return ret;
13 }
In terms of jumpiness, without the patch, the jump sequence is:
2 -> 13 -> 4 -> 11 -> 13
With the patch, the jump sequence is:
2-> 9 -> 4 -> 11 -> 13
I think the patch does not increase the jumpiness, while it
significantly improves coverage.
Thanks,
Dehao