[5] Fix build issue with NO_FUNCTION_CSE

Message ID 5877BF16.4090004@foss.arm.com
State New
Headers show

Commit Message

Kyrill Tkachov Jan. 12, 2017, 5:38 p.m.
Hi all,

This patch fixes the build issue on x86_64 (and other targets that define NO_FUNCTION_CSE but not a particular value)
on the GCC 5 branch.

Tested on x86_64.
Committing to the branch in the interest of fixing the build.

Thanks,
Kyrill

2016-01-12  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * postreload.c (reload_cse_simplify): Check for NO_FUNCTION_CSE
     definition instead of using it directly.

Comments

Nathan Sidwell Jan. 12, 2017, 8:57 p.m. | #1
On 01/12/2017 12:38 PM, Kyrill Tkachov wrote:
> Hi all,

>

> This patch fixes the build issue on x86_64 (and other targets that

> define NO_FUNCTION_CSE but not a particular value)

> on the GCC 5 branch.


thanks!


-- 
Nathan Sidwell

Patch hide | download patch | download mbox

diff --git a/gcc/postreload.c b/gcc/postreload.c
index ba6591d..fc0499a 100644
--- a/gcc/postreload.c
+++ b/gcc/postreload.c
@@ -122,8 +122,10 @@  reload_cse_simplify (rtx_insn *insn, rtx testreg)
 
   /* If NO_FUNCTION_CSE has been set by the target, then we should not try
      to cse function calls.  */
-  if (NO_FUNCTION_CSE && CALL_P (insn))
+#ifdef NO_FUNCTION_CSE
+  if (CALL_P (insn))
     return false;
+#endif
 
   if (GET_CODE (body) == SET)
     {