Fix usage of GETPC(), variable shadowing

[PATCH 0/2] Fix usage of GETPC(), variable shadowing

Brian Cain posted 2 patches 7 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231004123957.1732915-1-bcain@quicinc.com

Maintainers: Brian Cain <bcain@quicinc.com>

target/hexagon/imported/alu.idef |  6 +--
target/hexagon/macros.h          | 19 ++++----
target/hexagon/mmvec/macros.h    |  2 +-
target/hexagon/op_helper.c       | 84 ++++++++++++--------------------
target/hexagon/op_helper.h       |  9 ----
target/hexagon/translate.c       | 10 ++--
6 files changed, 50 insertions(+), 80 deletions(-)

Expand all Fold all

[PATCH 0/2] Fix usage of GETPC(), variable shadowing

Posted by Brian Cain 7 months ago

Matheus' patch has previously been reviewed, but I based my -Wshadow
patch on his.  So I'm submitting the series for review.

Brian Cain (1):
  target/hexagon: fix some occurrences of -Wshadow=local

Matheus Tavares Bernardino (1):
  target/hexagon: move GETPC() calls to top level helpers

 target/hexagon/imported/alu.idef |  6 +--
 target/hexagon/macros.h          | 19 ++++----
 target/hexagon/mmvec/macros.h    |  2 +-
 target/hexagon/op_helper.c       | 84 ++++++++++++--------------------
 target/hexagon/op_helper.h       |  9 ----
 target/hexagon/translate.c       | 10 ++--
 6 files changed, 50 insertions(+), 80 deletions(-)

-- 
2.25.1

Re: [PATCH 0/2] Fix usage of GETPC(), variable shadowing

Posted by Markus Armbruster 7 months ago

Brian Cain <bcain@quicinc.com> writes:

> Matheus' patch has previously been reviewed, but I based my -Wshadow
> patch on his.  So I'm submitting the series for review.

I'm not queueing these, because it's more than just shadowing.  I trust
your PR will arrive in due time :)

Thanks!

Re: [PATCH 0/2] Fix usage of GETPC(), variable shadowing

Posted by Anton Johansson via 7 months ago

On 04/10/23, Brian Cain wrote:
> Matheus' patch has previously been reviewed, but I based my -Wshadow
> patch on his.  So I'm submitting the series for review.
> 
> Brian Cain (1):
>   target/hexagon: fix some occurrences of -Wshadow=local
> 
> Matheus Tavares Bernardino (1):
>   target/hexagon: move GETPC() calls to top level helpers
> 
>  target/hexagon/imported/alu.idef |  6 +--
>  target/hexagon/macros.h          | 19 ++++----
>  target/hexagon/mmvec/macros.h    |  2 +-
>  target/hexagon/op_helper.c       | 84 ++++++++++++--------------------
>  target/hexagon/op_helper.h       |  9 ----
>  target/hexagon/translate.c       | 10 ++--
>  6 files changed, 50 insertions(+), 80 deletions(-)
> 
> -- 
> 2.25.1
> 

Both patches:
Reviewed-by: Anton Johansson <anjo@rev.ng>
Tested-by: Anton Johansson <anjo@rev.ng>