On Mon, Aug 27, 2018 at 9:13 AM Martin Liška <mli...@suse.cz> wrote:
>
> Hi.
>
> There's updated version with notes that were discussed with Richi
> on IRC. As pointed out label_to_block can potentially lead to
> quadratic complexity, but it's not ambition of the patch to resolve
> that.

Please don't

@@ -3239,7 +3239,7 @@ expand_asm_stmt (gasm *stmt)
             may insert further instructions into the same basic block after
             asm goto and if we don't do this, insertion of instructions on
             the fallthru edge might misbehave.  See PR58670.  */
-         if (fallthru_bb && label_to_block_fn (cfun, label) == fallthru_bb)
+         if (fallthru_bb && label_to_block (label) == fallthru_bb)
            {
              if (fallthru_label == NULL_RTX)
                fallthru_label = gen_label_rtx ();

rather the label_to_block function should go away.  OK with removing any
such instances in the patch.

Richard.

> Martin

Reply via email to