Quoting Jani Nikula (2019-03-21 09:50:25)
> On Wed, 20 Mar 2019, Chris Wilson <ch...@chris-wilson.co.uk> wrote:
> > gcc-4.8 and older dislike the use of __builtin_constant_p() within a
> > constant expression context, and so we must use the magical
> > __is_constexpr() instead.
> >
> > For example, with gcc-4.8.5:
> > ../drivers/gpu/drm/i915/i915_reg.h:167:27: error: first argument to 
> > ‘__builtin_choose_expr’ not a constant
> > ../include/linux/build_bug.h:16:45: error: bit-field ‘<anonymous>’ width 
> > not an integer constant
> >
> > Reported-by: Randy Dunlap <rdun...@infradead.org>
> > Fixes: baa09e7d2f42 ("drm/i915: use REG_FIELD_PREP() to define register 
> > bitfield values")
> > Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> > Cc: Jani Nikula <jani.nik...@intel.com>
> > Cc: Imre Deak <imre.d...@intel.com>
> > Cc: Randy Dunlap <rdun...@infradead.org>
> 
> Thanks, Uma reported this too and confirmed __is_constexpr works.
> 
> Reported-by: Uma Shankar <uma.shan...@intel.com>
> Reviewed-by: Jani Nikula <jani.nik...@intel.com>

And pushed for a quieter tomorrow. Thanks everyone for reporting!
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to