[PATCH] D23557: [RISCV 1/10] Recognise riscv32 and riscv64 in triple parsing code

James Y Knight via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 13:14:03 PDT 2016


jyknight added a comment.

Before this can be committed to llvm, permission to contribute from everyone who's committed code to the upstream branch will need to be collected.

git log --format="%an <%ae>" llvm/master..riscv-llvm/riscv-trunk|sort |uniq -c
shows, it looks like, 11 people whose permission needs to be asked. Some of those only contributed a 1-line trivial fix, so possibly it's okay if they don't reply, but might as well try asking everyone first. Certainly the more major contributors need to say OK.

I believe that if you cc those authors on the proposal email for adding the backend to LLVM, and ask them to explicitly reply to the list that they're okay with their work in https://github.com/riscv/riscv-llvm getting contributed to LLVM, under the LLVM license, that will be sufficient.

It'd also be a good idea to put the names/emails of the relevant additional original authors in each patch.


https://reviews.llvm.org/D23557





More information about the llvm-commits mailing list