Re: [Mesa-dev] [PATCH] PowerPC: Altivec IROUND operation

2013-04-02 Thread Jose Fonseca
I don't see need/benefit in mixing "iround" (ie, float -> int) with "round" (ie, float -> float). If this is a one-off, then you should just call lp_build_intrinsic_unary(builder, "llvm.ppc.altivec.vctsxs", ...) If you really need an generic intrinsic helper for iround, then please add a new

[Mesa-dev] [PATCH] PowerPC: Altivec IROUND operation

2013-04-01 Thread Adam Jackson
From: Adhemerval Zanella This adds another rounding mode to the enum, which happens otherwise to match SSE4.1's rounding modes. This should be safe as long as the IROUND case never hits the SSE4.1 path. Reviewed-by: Adam Jackson Signed-off-by: Adhemerval Zanella --- src/gallium/auxiliary/gal