Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-10 Thread Chandler Carruth via cfe-commits 
chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

Danny's OK is enough for me. LGTM


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-10 Thread Aaron Ballman via cfe-commits 
aaron.ballman closed this revision.
aaron.ballman added a comment.

Commit in r255248, thank you for the review!

~Aaron


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-08 Thread Aaron Ballman via cfe-commits 
aaron.ballman added a comment.

Ping


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-08 Thread Daniel Berlin via cfe-commits 
dberlin added a comment.

The license  looks fine to me


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-08 Thread Alexander Kornienko via cfe-commits 
alexfh added a comment.

Daniel, is the proposed solution fine?


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits