[llvm-commits] [PATCH][Target/PTX] Add lower return and argument
    Rafael Espindola 
    espindola at google.com
       
    Wed Oct 13 07:46:56 PDT 2010
    
    
  
On 12 October 2010 23:24, Che-Liang Chiou <clchiou at gmail.com> wrote:
> Re-send this patch in case no one noticed.
I am not familiar with PTX, but since this patch is all in that
backend, it is probably OK. I think my only comment is that some of
the asserts should be errors. For example, varargs are not support or
just not implemented yet? If not supported it should probably be an
error. That way you still get an error on builds without assertions.
Cheers,
-- 
Rafael Ávila de Espíndola
    
    
More information about the llvm-commits
mailing list