Re: [PATCH] D27123: Add AVR target and toolchain to Clang

2017-01-04 Thread Dylan McKay via cfe-commits
I've just raised a bug here https://llvm.org/bugs/show_bug.cgi?id=31530 > What's necessary for sign-off? Should I ping the reviewer (Richard Smith) again? I think so long as Richard or someone else who actively works on Clang signs off. Perhaps somebody else is keen to review? On the other hand

Re: [PATCH] D27123: Add AVR target and toolchain to Clang

2017-01-04 Thread Dylan McKay via cfe-commits
Did you get the BugZilla account sorted Senthil? On Thu, Dec 22, 2016 at 10:47 AM, Nico Weber wrote: > On Wed, Dec 21, 2016 at 6:42 AM, Senthil Kumar Selvaraj via Phabricator > via cfe-commits wrote: > >> saaadhu added a comment. >> >> Would someone with a llvm bugzilla account please file the

Re: [PATCH] D27123: Add AVR target and toolchain to Clang

2017-01-04 Thread Jonathan Roelofs via cfe-commits
On 1/4/17 2:52 AM, Dylan McKay wrote: I've just raised a bug here https://llvm.org/bugs/show_bug.cgi?id=31530 > What's necessary for sign-off? Should I ping the reviewer (Richard Smith) again? I think so long as Richard or someone else who actively works on Clang signs off. Perhaps somebo

Re: [PATCH] D27123: Add AVR target and toolchain to Clang

2017-01-03 Thread Senthil Kumar Selvaraj via cfe-commits
Dylan McKay writes: > Did you get the BugZilla account sorted Senthil? Nope, direct email also didn't help. Can you please file a bug with Title: UINT16_TYPE and INT16_TYPE are defined as short instead of int for AVR Description: UINT16_TYPE and INT16_TYPE are implicitly defined by the pr

Re: [PATCH] D27123: Add AVR target and toolchain to Clang

2016-12-21 Thread Nico Weber via cfe-commits
On Wed, Dec 21, 2016 at 6:42 AM, Senthil Kumar Selvaraj via Phabricator via cfe-commits wrote: > saaadhu added a comment. > > Would someone with a llvm bugzilla account please file the PR for me? New > user registration is disabled, and I've been waiting for > llvm-ad...@lists.llvm.org to respond

Re: [PATCH] D27123: Add AVR target and toolchain to Clang

2016-12-21 Thread Dylan McKay via cfe-commits
I'm happy to file a bug if you give me a description > Would one of the reviewers commit the patch for me? I'm happy to commit it, so long as somebody else signs off on it. On Thu, Dec 22, 2016 at 12:42 AM, Senthil Kumar Selvaraj via Phabricator < revi...@reviews.llvm.org> wrote: > saaadhu adde