On Wed, 19 Sep 2012, Chris Wilson <[email protected]> wrote: > On Wed, 19 Sep 2012 11:12:41 +1000, Dave Airlie <[email protected]> wrote: >> From: Dave Airlie <[email protected]> >> >> We hit this a lot with i915 and although we'd like to engineer things to hit >> it a lot less, this commit at least makes it consume a few less cycles. >> >> from something containing >> movzwl 0x0(%rip),%r10d >> to >> add %r8,%rdx >> >> I only noticed it while using perf to profile something else. >> >> Signed-off-by: Dave Airlie <[email protected]> > Reviewed-by: Chris Wilson <[email protected]> > > I would have thought this is something the compiler should, with the > appropriate hints, be smart enough to decide for itself as boot_cpu_data > should be constant once defined. *shrug*
Since it's not obvious the compiler isn't smart enough, should the code carry a comment so we don't forget? BR, Jani. _______________________________________________ dri-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/dri-devel
