On 24.12.2014 02:49, Tom Stellard wrote: > Rather than building a new one every compile. This should reduce some > of the overhead of compiling shaders.
Thanks, though unfortunately it doesn't seem to make much difference for piglit for me. > One consequence of this change is that we lose the MachineInstrs dumps > when dumping the shaders via R600_DEBUG. The LLVM IR and assembly is > still dumped, and if you still want to see the MachineInstr dump, you > can run the dumped LLVM IR through llc. Sounds reasonable, but... > @@ -503,5 +510,12 @@ struct pipe_screen *radeonsi_screen_create(struct > radeon_winsys *ws) > /* Create the auxiliary context. This must be done last. */ > sscreen->b.aux_context = sscreen->b.b.context_create(&sscreen->b.b, > NULL); > > + /* Initialize LLVM TargetMachine */ > + r600_target = radeon_llvm_get_r600_target(); > + sscreen->tm = LLVMCreateTargetMachine(r600_target, "r600--", > + r600_get_llvm_processor_name(sscreen->b.family), > + "+DumpCode", LLVMCodeGenLevelDefault, > LLVMRelocDefault, > + LLVMCodeModelDefault); > + > return &sscreen->b.b; > } ... since you pass "+DumpCode" here, the MachineInstrs are actually always dumped unconditionally. With that fixed, this patch is Reviewed-by: Michel Dänzer <michel.daen...@amd.com> And so is patch 2. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
