[Mesa-dev] [PATCH] R600/SI: Add pattern for fp_to_uint

Tom Stellard tom at stellard.net
Tue Jul 30 09:02:45 PDT 2013


On Tue, Jul 30, 2013 at 05:54:58PM +0200, Michel Dänzer wrote:
> On Die, 2013-07-30 at 07:47 -0700, Tom Stellard wrote:
> > 
> > You will need to include a lit test with this patch.
> 
> Ah yes, this occurred to me after my review. :)
> 
> 
> > Also, if you try to run piglit with a debug version of LLVM, it will
> > take forever.  I always install a release version of llvm for testing
> > and running piglit, and if I need to debug an individual test, I build
> > (but don't install) a debug version of LLVM using:
> > 
> > make ENABLE_OPTIMIZED=0
> > 
> > and then use LD_PRELOAD=llvm/Debug+Asserts/lib/libLLVM-3.4svn.so some_app
> > for testing.
> 
> FWIW, I'm building LLVM with '--enable-optimized
> --with-optimize-option=-O2 --enable-assertions --enable-debug-runtime
> --enable-debug-symbols', which has been fast enough for piglit (less
> than 10 minutes for quick-driver.tests) and good enough for debugging
> for me. YMMV.
>

Oh nice, I wasn't aware of those options.  I'll have to try that.

-Tom
> Also, building LLVM with --enable-shared will save a significant amount
> of time for linking in Mesa (and possibly LLVM as well).
> 
> 
> -- 
> Earthling Michel Dänzer           |                   http://www.amd.com
> Libre software enthusiast         |          Debian, X and DRI developer




More information about the llvm-commits mailing list