[Bug sanitizer/64981] New: [5.0 regression] -fsanitize=address cant expand __builtin_ia32_rdtsc

[Bug sanitizer/64981] New: [5.0 regression] -fsanitize=address cant expand __builtin_ia32_rdtsc

[dimhen at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64981

            Bug ID: 64981
           Summary: [5.0 regression] -fsanitize=address cant expand
                    __builtin_ia32_rdtsc
           Product: gcc
           Version: 5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: sanitizer
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dimhen at gmail dot com
                CC: dodji at gcc dot gnu.org, dvyukov at gcc dot gnu.org,
                    jakub at gcc dot gnu.org, kcc at gcc dot gnu.org

r216735 PASS
r216827 FAIL
r220528 FAIL

$ cat a.c
int main()
{
    __builtin_ia32_rdtsc();
    return 0;
}

$ /usr/local/gcc_current/bin/gcc -fsanitize=address -S a.c && grep rdtsc a.s
    call    __builtin_ia32_rdtsc

$ ~/bin/gcc_216735/bin/gcc -fsanitize=address -S a.c && grep rdtsc a.s
    rdtsc

$ ~/bin/gcc_216827/bin/gcc -fsanitize=address -S a.c && grep rdtsc a.s
    call    __builtin_ia32_rdtsc

[Bug sanitizer/64981] [5.0 regression] -fsanitize=address cant expand __builtin_ia32_rdtsc

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64981

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2015-02-09
           Assignee|unassigned at gcc dot gnu.org      |jakub at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 34704
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34704&action=edit
gcc5-pr64981.patch

Untested fix.

[Bug sanitizer/64981] [5 regression] -fsanitize=address cant expand __builtin_ia32_rdtsc

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64981

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |5.0
            Summary|[5.0 regression]            |[5 regression]
                   |-fsanitize=address cant     |-fsanitize=address cant
                   |expand __builtin_ia32_rdtsc |expand __builtin_ia32_rdtsc

[Bug sanitizer/64981] [5 regression] -fsanitize=address cant expand __builtin_ia32_rdtsc

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64981

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Mon Feb  9 21:23:58 2015
New Revision: 220551

URL: https://gcc.gnu.org/viewcvs?rev=220551&root=gcc&view=rev
Log:
    PR sanitizer/64981
    * builtins.c (expand_builtin): Call targetm.expand_builtin
    for BUILT_IN_MD builtins regardless of asan_intercepted_p.

    * gcc.dg/asan/pr64981.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/asan/pr64981.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/builtins.c
    trunk/gcc/testsuite/ChangeLog

[Bug sanitizer/64981] [5 regression] -fsanitize=address cant expand __builtin_ia32_rdtsc

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64981

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.