On Monday, June 11, 2018 7:26:11 PM PDT Francisco Jerez wrote:
> diff --git a/src/intel/compiler/brw_eu.c b/src/intel/compiler/brw_eu.c
> index 6ef0a6a577c..80f05240b42 100644
> --- a/src/intel/compiler/brw_eu.c
> +++ b/src/intel/compiler/brw_eu.c
> @@ -356,14 +356,14 @@ brw_disassemble(const struct gen_device_info *devinfo,
> bool dump_hex = (INTEL_DEBUG & DEBUG_HEX) != 0;
>
> for (int offset = start; offset < end;) {
> - const brw_inst *insn = assembly + offset;
> + const brw_inst *insn = (const brw_inst *)((char *)assembly + offset);This isn't standard C either, but void * math is a pretty common compiler extension that's been around forever. Personally, I'd prefer that we keep using it, as it's much more readable.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
