[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