| Message ID | 20201216180528.187152-1-richard.henderson@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | tcg: Add tcg_gen_bswap_tl alias | expand |
On Thu, Dec 17, 2020 at 2:05 AM Richard Henderson < richard.henderson@linaro.org> wrote: > The alias is intended to indicate that the bswap is for the > entire target_long. This should avoid ifdefs on some targets. > > Signed-off-by: Richard Henderson <richard.henderson@linaro.org> > --- > include/tcg/tcg-op.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/tcg/tcg-op.h b/include/tcg/tcg-op.h > index 5abf17fecc..5b3bdacc39 100644 > --- a/include/tcg/tcg-op.h > +++ b/include/tcg/tcg-op.h > @@ -1085,6 +1085,7 @@ void tcg_gen_stl_vec(TCGv_vec r, TCGv_ptr base, > TCGArg offset, TCGType t); > #define tcg_gen_bswap16_tl tcg_gen_bswap16_i64 > #define tcg_gen_bswap32_tl tcg_gen_bswap32_i64 > #define tcg_gen_bswap64_tl tcg_gen_bswap64_i64 > +#define tcg_gen_bswap_tl tcg_gen_bswap64_i64 > #define tcg_gen_concat_tl_i64 tcg_gen_concat32_i64 > #define tcg_gen_extr_i64_tl tcg_gen_extr32_i64 > #define tcg_gen_andc_tl tcg_gen_andc_i64 > @@ -1197,6 +1198,7 @@ void tcg_gen_stl_vec(TCGv_vec r, TCGv_ptr base, > TCGArg offset, TCGType t); > #define tcg_gen_ext32s_tl tcg_gen_mov_i32 > #define tcg_gen_bswap16_tl tcg_gen_bswap16_i32 > #define tcg_gen_bswap32_tl tcg_gen_bswap32_i32 > +#define tcg_gen_bswap_tl tcg_gen_bswap32_i32 > #define tcg_gen_concat_tl_i64 tcg_gen_concat_i32_i64 > #define tcg_gen_extr_i64_tl tcg_gen_extr_i64_i32 > #define tcg_gen_andc_tl tcg_gen_andc_i32 > -- > 2.25.1 > > Thanks, I'll apply this one to my RISC-V B-extension patchset. Reviewed-by: Frank Chang <frank.chang@sifive.com> <div dir="ltr"><div dir="ltr">On Thu, Dec 17, 2020 at 2:05 AM Richard Henderson <<a href="mailto:richard.henderson@linaro.org">richard.henderson@linaro.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The alias is intended to indicate that the bswap is for the<br> entire target_long. This should avoid ifdefs on some targets.<br> <br> Signed-off-by: Richard Henderson <<a href="mailto:richard.henderson@linaro.org" target="_blank">richard.henderson@linaro.org</a>><br> ---<br> include/tcg/tcg-op.h | 2 ++<br> 1 file changed, 2 insertions(+)<br> <br> diff --git a/include/tcg/tcg-op.h b/include/tcg/tcg-op.h<br> index 5abf17fecc..5b3bdacc39 100644<br> --- a/include/tcg/tcg-op.h<br> +++ b/include/tcg/tcg-op.h<br> @@ -1085,6 +1085,7 @@ void tcg_gen_stl_vec(TCGv_vec r, TCGv_ptr base, TCGArg offset, TCGType t);<br> #define tcg_gen_bswap16_tl tcg_gen_bswap16_i64<br> #define tcg_gen_bswap32_tl tcg_gen_bswap32_i64<br> #define tcg_gen_bswap64_tl tcg_gen_bswap64_i64<br> +#define tcg_gen_bswap_tl tcg_gen_bswap64_i64<br> #define tcg_gen_concat_tl_i64 tcg_gen_concat32_i64<br> #define tcg_gen_extr_i64_tl tcg_gen_extr32_i64<br> #define tcg_gen_andc_tl tcg_gen_andc_i64<br> @@ -1197,6 +1198,7 @@ void tcg_gen_stl_vec(TCGv_vec r, TCGv_ptr base, TCGArg offset, TCGType t);<br> #define tcg_gen_ext32s_tl tcg_gen_mov_i32<br> #define tcg_gen_bswap16_tl tcg_gen_bswap16_i32<br> #define tcg_gen_bswap32_tl tcg_gen_bswap32_i32<br> +#define tcg_gen_bswap_tl tcg_gen_bswap32_i32<br> #define tcg_gen_concat_tl_i64 tcg_gen_concat_i32_i64<br> #define tcg_gen_extr_i64_tl tcg_gen_extr_i64_i32<br> #define tcg_gen_andc_tl tcg_gen_andc_i32<br> -- <br> 2.25.1<br> <br></blockquote><div><br></div><div>Thanks, I'll apply this one to my RISC-V B-extension patchset.</div><div><br></div><div>Reviewed-by: Frank Chang <<a href="mailto:frank.chang@sifive.com">frank.chang@sifive.com</a>> </div></div></div>
diff --git a/include/tcg/tcg-op.h b/include/tcg/tcg-op.h index 5abf17fecc..5b3bdacc39 100644 --- a/include/tcg/tcg-op.h +++ b/include/tcg/tcg-op.h @@ -1085,6 +1085,7 @@ void tcg_gen_stl_vec(TCGv_vec r, TCGv_ptr base, TCGArg offset, TCGType t); #define tcg_gen_bswap16_tl tcg_gen_bswap16_i64 #define tcg_gen_bswap32_tl tcg_gen_bswap32_i64 #define tcg_gen_bswap64_tl tcg_gen_bswap64_i64 +#define tcg_gen_bswap_tl tcg_gen_bswap64_i64 #define tcg_gen_concat_tl_i64 tcg_gen_concat32_i64 #define tcg_gen_extr_i64_tl tcg_gen_extr32_i64 #define tcg_gen_andc_tl tcg_gen_andc_i64 @@ -1197,6 +1198,7 @@ void tcg_gen_stl_vec(TCGv_vec r, TCGv_ptr base, TCGArg offset, TCGType t); #define tcg_gen_ext32s_tl tcg_gen_mov_i32 #define tcg_gen_bswap16_tl tcg_gen_bswap16_i32 #define tcg_gen_bswap32_tl tcg_gen_bswap32_i32 +#define tcg_gen_bswap_tl tcg_gen_bswap32_i32 #define tcg_gen_concat_tl_i64 tcg_gen_concat_i32_i64 #define tcg_gen_extr_i64_tl tcg_gen_extr_i64_i32 #define tcg_gen_andc_tl tcg_gen_andc_i32
The alias is intended to indicate that the bswap is for the entire target_long. This should avoid ifdefs on some targets. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> --- include/tcg/tcg-op.h | 2 ++ 1 file changed, 2 insertions(+) -- 2.25.1