re PR sanitizer/64981 (-fsanitize=address cant expand __builtin_ia32_rdtsc)

	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.

From-SVN: r220551

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 2791de6c4f86919ec512e12431b5c7b5c8b64ebd..328c37d030398b3080dc7f1ca6d04d7935413513 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2015-02-09  Jakub Jelinek  <jakub@redhat.com>
+
+       PR sanitizer/64981
+       * builtins.c (expand_builtin): Call targetm.expand_builtin
+       for BUILT_IN_MD builtins regardless of asan_intercepted_p.
+
 2015-02-08  Jan Hubicka  <hubicka@ucw.cz>
 
        PR ipa/61548

diff --git a/gcc/builtins.c b/gcc/builtins.c
index 3ddaddc1ab061815f3a0f12d2850baca6b9deb7a..8f3c0bc22b30541310b1bfa453820cfee2341b0d 100644 (file)
--- a/gcc/builtins.c
+++ b/gcc/builtins.c
@@ -5960,6 +5960,9 @@ expand_builtin (tree exp, rtx target, rtx subtarget, machine_mode mode,
   machine_mode target_mode = TYPE_MODE (TREE_TYPE (exp));
   int flags;
 
+  if (DECL_BUILT_IN_CLASS (fndecl) == BUILT_IN_MD)
+    return targetm.expand_builtin (exp, target, subtarget, mode, ignore);
+
   /* When ASan is enabled, we don't want to expand some memory/string
      builtins and rely on libsanitizer's hooks.  This allows us to avoid
      redundant checks and be sure, that possible overflow will be detected
@@ -5968,9 +5971,6 @@ expand_builtin (tree exp, rtx target, rtx subtarget, machine_mode mode,
   if ((flag_sanitize & SANITIZE_ADDRESS) && asan_intercepted_p (fcode))
     return expand_call (exp, target, ignore);
 
-  if (DECL_BUILT_IN_CLASS (fndecl) == BUILT_IN_MD)
-    return targetm.expand_builtin (exp, target, subtarget, mode, ignore);
-
   /* When not optimizing, generate calls to library functions for a certain
      set of builtins.  */
   if (!optimize

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 3d14e6041b802f24c262a90e31867e62260778cb..67f1e8e5355109d1d1ebe1184357647f64318af2 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2015-02-09  Jakub Jelinek  <jakub@redhat.com>
+
+       PR sanitizer/64981
+       * gcc.dg/asan/pr64981.c: New test.
+
 2015-02-09  Marek Polacek  <polacek@redhat.com>
 
        PR c/64856

diff --git a/gcc/testsuite/gcc.dg/asan/pr64981.c b/gcc/testsuite/gcc.dg/asan/pr64981.c
new file mode 100644 (file)
index 0000000..85e3479
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/asan/pr64981.c
@@ -0,0 +1,12 @@
+/* PR sanitizer/64981 */
+/* { dg-do compile { target { i?86-*-* x86_64-*-* } } } */
+/* { dg-options "-fsanitize=address -march=x86-64" } */
+
+int
+main ()
+{
+  __builtin_ia32_rdtsc ();
+  return 0;
+}
+
+/* { dg-final { scan-assembler-not "__builtin_ia32_rdtsc" } } */