https://bugs.kde.org/show_bug.cgi?id=423195

--- Comment #24 from Julian Seward <jsew...@acm.org> ---
(In reply to Carl Love from comment #15)
> Created attachment 130632 [details]
> Add prefixed support for the following word

ok to land but please at least consider the following:

> +/* Prefix instsruction effective address calc: (rA + simm) */

typo: instruction

> +   /* Get the EA */
> +   if ( *R == 0 )
> +      return mkexpr ( tmp );

At least in a simplistic reading of this function, it looks like *R is read on
all paths, but only written on the `prefix` path.  So is it defined before the
call, in that case?

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to