[Pixman] [PATCH] SIMD: Try without any CFLAGS before forcing -mcpu=

Loïc Minier lool at dooz.org
Sun Mar 14 10:25:10 PDT 2010


On Wed, Mar 10, 2010, Siarhei Siamashka wrote:
> I would prefer a bit more descriptive comment (with the details copied
> from that launchpad page).

 I see you pushed this now; thanks!  Yeah, it's not obvious why one
 needs to try with the toolchain defaults first.  I'm attaching a patch
 to update the comments.

> Patch looks mostly OK to me and improves arm simd support a bit. It still
> fails with CFLAGS="-O2 -mthumb" (and toolchain configured to compile for armv4
> by default)

 This is unfortunate, but even if I could change the configure test to
 succeed here, the actual build would place CFLAGS last
 (user-overrides); it's perhaps best if the build fails at this point.
 The other option would be to munge the user-specific CFLAGS in
 configure, but I'm not sure I'm confortable with that, plus these can
 be overriden during the build as well.

 So I guess people forcing Thumb in CFLAGS with a default of an old CPU
 will have a bit more pain here.  I think it also fails if you force
 -mcpu=oldcpu in CFLAGS.

> But that was also happening earlier, so it is not a regression.

 Ack

> In my opinion the best solution overall would be to move all the assembly
> optimizations into separate .S files also for legacy ARM processors and
> get rid of these compiler option hacks. I think that bringing support for
> legacy ARM processors into a better shape is quite realistic even for 0.18.0
> stable release, which is due to be released this month. But it can only happen
> if enough people are interested in this, and more importantly, are ready to
> actively participate in testing.

 I think this should be kept as an open bug against pixman that the
 inline asm()s would better be written as separate .S files (thanks for
 the idea).

-- 
Loïc Minier


More information about the Pixman mailing list