On 4/4/25 19:53, Philippe Mathieu-Daudé wrote:
On 4/4/25 18:48, Pierrick Bouvier wrote:
On 4/3/25 16:49, Philippe Mathieu-Daudé wrote:
Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
---
  target/tricore/translate.c | 6 ++++--
  1 file changed, 4 insertions(+), 2 deletions(-)


The temptation is good, but please do not touch any target code at this point. We want to focus on defining the API first, and we can perform codebase changes as a second step, without letting any occurrences of the old macros/functions, instead of just adding "another way to do it".

I meant to remove these patch before posting, to focus on ARM, but
apparently forgot to do so...

In this particular case TARGET_LONG_BITS is a constant 32 (and 32 is present many times in the code), so it can be replaced as a cleanup.

Paolo


Reply via email to