Only resolve_asm_operand_names once

We do it in the front end already; no need to repeat.

From-SVN: r223022

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d7d5ca01cfec70493f686ae454a0bc0dfb752554..98394eee61bcb441826641f5bcd5d0285981a8ff 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2015-05-11  Richard Henderson  <rth@redhat.com>
+
+       * cfgexpand.c (expand_asm_operands): Don't call
+       resolve_asm_operand_names.
+       * stmt.c (resolve_asm_operand_names): Clarify block comment.
+
 2015-05-11  Jan Hubicka  <hubicka@ucw.cz>
 
        * dwarf2out.c (gen_member_die): Sanity check that we access

diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c
index b76a219c4f4f232a20c65157471ed1d1928022ce..69d00a00095c3d00644118f4aacc87f74ad638bc 100644 (file)
--- a/gcc/cfgexpand.c
+++ b/gcc/cfgexpand.c
@@ -2554,8 +2554,6 @@ expand_asm_operands (tree string, tree outputs, tree inputs,
   if (! check_operand_nalternatives (outputs, inputs))
     return;
 
-  string = resolve_asm_operand_names (string, outputs, inputs, labels);
-
   /* Collect constraints.  */
   i = 0;
   for (t = outputs; t ; t = TREE_CHAIN (t), i++)

diff --git a/gcc/stmt.c b/gcc/stmt.c
index 6c62a129601ccb65d5f7cb289c197d5e653003d8..e63179f97760e1e53e0a658ff287269c3220b08a 100644 (file)
--- a/gcc/stmt.c
+++ b/gcc/stmt.c
@@ -562,9 +562,10 @@ check_unique_operand_names (tree outputs, tree inputs, tree labels)
   return false;
 }
 
-/* A subroutine of expand_asm_operands.  Resolve the names of the operands
-   in *POUTPUTS and *PINPUTS to numbers, and replace the name expansions in
-   STRING and in the constraints to those numbers.  */
+/* Resolve the names of the operands in *POUTPUTS and *PINPUTS to numbers,
+   and replace the name expansions in STRING and in the constraints to
+   those numbers.  This is generally done in the front end while creating
+   the ASM_EXPR generic tree that eventually becomes the GIMPLE_ASM.  */
 
 tree
 resolve_asm_operand_names (tree string, tree outputs, tree inputs, tree labels)