On 12/14/20 2:23 PM, Rebecca Cran wrote: > +++ b/target/arm/cpu64.c > @@ -620,6 +620,7 @@ static void aarch64_max_initfn(Object *obj) > t = FIELD_DP64(t, ID_AA64ISAR0, DP, 1); > t = FIELD_DP64(t, ID_AA64ISAR0, FHM, 1); > t = FIELD_DP64(t, ID_AA64ISAR0, TS, 2); /* v8.5-CondM */ > + t = FIELD_DP64(t, ID_AA64ISAR0, TLB, 2); > t = FIELD_DP64(t, ID_AA64ISAR0, RNDR, 1);
When the value is not 1, we have generally added a comment to clarify exactly which extensions this covers. Though it's slightly annoying that F.c has renamed all of the extensions. With that added, Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~