Jiong Wang
2013-Mar-23 09:51 UTC
[LLVMdev] About commit TILE-Gx backend to community repository and default disabled
on 2013/3/23 1:52, Chris Lattner wrote:> On Mar 19, 2013, at 8:58 PM, Jiong Wang <jiwang at tilera.com> wrote: > >> Hi Chris, >> >> could you please comment on committing TILE-Gx backend into community? > Hi Jiong, > > I don't have any special advice here. It sounds like the general functionality level is high enough. Taking it into mainline sounds great, so long as it is reviewed by someone.thanks. I will commit after rebasing & re-testing the code on latest llvm mainline. And I will follow http://llvm.org/docs/HowToAddABuilder.html to setup a TILE-Gx buildbot. --- Regards, Jiong Tilera Corporation.> > -Chris > > >> >> ========== TILE-Gx Status ==========>> >> Features Supported >> ==>> 1. general function. >> 2. PIC/TLS/JumpTable. >> 3. Instructoin Bundling for VLIW. >> 4. Asm Parser >> 5. MC Layer (aware of instruction bundle), MCJIT support. >> 6. Initial regression tests for CodeGen & MC Layer. >> >> Regression Result >> ==>> Expected Passes : 13363 >> Expected Failures : 84 >> Unsupported Tests : 76 >> Unexpected Failures: 21 >> >> 20 of the unexpected failures are caused by lack of old jit support. >> >> Test-Suite Result >> ==>> Expected Passes : 953 >> Unexpected Failures: 14 >> (all 14 failures has the same output as tilegx gcc, >> most of them are about float precision issue) >> >> >> >> 于 2013/3/16 2:37, Tom Stellard 写道: >>> On Fri, Mar 15, 2013 at 05:38:28PM +0800, Jiong Wang wrote: >>>> Hi Chandler, >>>> >>>> on 2013/3/15 17:15, Chandler Carruth wrote: >>>>> On Fri, Mar 15, 2013 at 1:54 AM, Jiong Wang <jiwang at tilera.com >>>>> <mailto:jiwang at tilera.com>> wrote: >>>>> >>>>> I agree that everyone should contribute to keep the community >>>>> active and vigorious. But I think there are difference between >>>>> contributors. >>>>> >>>>> some are focused and with expertise on middle end, while others >>>>> may on front or back end. so the normal way for a new contributor >>>>> is, find a familiar point to start, then go deeper and wider and >>>>> expand one's contribution during this process. >>>>> >>>>> For Tilera corporation and me, we would like the keep tilegx >>>>> backend actively maintained and improved, and wish it could be a >>>>> good target to improve and test VLIW, many core features etc. >>>>> >>>>> >>>>> Inevitably, if you care about LLVM's support for your platform, >>>>> you will care about bugs and features in the target-independent >>>>> stack of LLVM's software (including the target independent parts >>>>> of Clang, etc). There will be bugs which are priorities only for >>>>> you and/or your users, features most pressing only for you or your >>>>> users, etc. As such, it seems likely that you will need at least a >>>>> basic working familiarity with the stack, and the ability to >>>>> improve it in at least basic ways. Simple things like finding and >>>>> fixing bugs in the target independent code generator, improving >>>>> DAG combine optimizations, or looking for compile time hotspots >>>>> are almost always approachable by nearly anyone on the project. >>>> I am not quite understand your points here, for a backend >>>> maintainer, of course, these things are daily work. >>>> >>>>> Essentially, you don't need to invent a new middle end >>>>> optimization, or improve the subtleties of our alias analysis. >>>>> Most of LLVM, and most of our bugs/missing features/areas for >>>>> improvement, involve straightforward C++ code and systems that >>>>> anyone who cares can go in, hack, and improve. That's what >>>>> occupies the majority of the maintenance work as it happens. There >>>>> is relatively little deeply detailed work requiring specialized >>>>> knowledge. >>>> Anyway, my question is "how could tilegx backend included in >>>> community repository?", I am here to seek answer for this. I >>>> searched mailinglist archive, and learned R600 and AArch64's request >>>> post, but find few things. >>>> >>> Hi Jiong, >>> >>> I am the maintainer of the R600 backend so I have some experience trying to >>> get a new backend included in the community repository. >>> >>> My recommendation is to contact the project lead, Chris Lattner, >>> directly and point him to the reviews that have already been done and ask >>> him specifically what you need to do to get the backend approved. >>> >>> There is no formal process in place for adding a new backend, and I >>> think if you keep posing your question to the list you will have trouble >>> getting a clear answer on what needs to be done >>> >>> -Tom >> _______________________________________________ >> LLVM Developers mailing list >> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu >> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
Tobias Grosser
2013-Mar-23 15:14 UTC
[LLVMdev] About commit TILE-Gx backend to community repository and default disabled
On 03/23/2013 10:51 AM, Jiong Wang wrote:> on 2013/3/23 1:52, Chris Lattner wrote: >> On Mar 19, 2013, at 8:58 PM, Jiong Wang <jiwang at tilera.com> wrote: >> >>> Hi Chris, >>> >>> could you please comment on committing TILE-Gx backend into community? >> Hi Jiong, >> >> I don't have any special advice here. It sounds like the general >> functionality level is high enough. Taking it into mainline sounds >> great, so long as it is reviewed by someone. > > thanks. > > I will commit after rebasing & re-testing the code on latest llvm > mainline. And I will follow http://llvm.org/docs/HowToAddABuilder.html > to setup a TILE-Gx buildbot.Hi Jiong, did you receive a full review by an LLVM committer who approved the full patch set (not just individual ones) with some kind of "Looks good to me"? This is the requirement Chris put for inclusion and only after this has happened (and there are no major concerns found during review) you should commit the back end. I am myself very supportive on getting this in, but I have the feeling this did not yet happen. So I propose to not commit this backend prematurely. (In case I missed something and this review already happened, would you mind pointing me to the email and committer who reviewed the backend) Thanks, Tobias
Jiong Wang
2013-Mar-23 15:32 UTC
[LLVMdev] About commit TILE-Gx backend to community repository and default disabled
äº 2013/3/23 23:14, Tobias Grosser åé:> On 03/23/2013 10:51 AM, Jiong Wang wrote: >> on 2013/3/23 1:52, Chris Lattner wrote: >>> On Mar 19, 2013, at 8:58 PM, Jiong Wang <jiwang at tilera.com> wrote: >>> >>>> Hi Chris, >>>> >>>> could you please comment on committing TILE-Gx backend into community? >>> Hi Jiong, >>> >>> I don't have any special advice here. It sounds like the general >>> functionality level is high enough. Taking it into mainline sounds >>> great, so long as it is reviewed by someone. >> >> thanks. >> >> I will commit after rebasing & re-testing the code on latest llvm >> mainline. And I will follow http://llvm.org/docs/HowToAddABuilder.html >> to setup a TILE-Gx buildbot. > > Hi Jiong, > > did you receive a full review by an LLVM committer who approved the > full patch set (not just individual ones) with some kind of "Looks > good to me"? This is the requirement Chris put for inclusion and only > after this has happened (and there are no major concerns found during > review) you should commit the back end. > > I am myself very supportive on getting this in, but I have the feeling > this did not yet happen. So I propose to not commit this backend > prematurely. (In case I missed something and this review already > happened, would you mind pointing me to the email and committer who > reviewed the backend)Hi Tobias, thanks for your reply, it's happy to receive more feedbacks. below are preivous community feedbacks, ==my first post to llvm-commit mailing list: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130304/167737.html my second post to llvm-commit mailing list which split the big patch into 17 small parts: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130311/167891.html Anton Korobeynikov, Jakob Stoklund Olesen, Joerg Sonnenberger given some feedback on coding style etc, all fixed. do you think it's OK ? please feel free to point out if there is anything missing. thank. --- Regards, Jiong Tilera Corporation.> > Thanks, > Tobias >
Chandler Carruth
2013-Mar-25 04:11 UTC
[LLVMdev] About commit TILE-Gx backend to community repository and default disabled
On Sun, Mar 24, 2013 at 7:30 PM, Jiong Wang <kasulle at gmail.com> wrote:> on 2013/3/23 1:52, Chris Lattner wrote: > >> On Mar 19, 2013, at 8:58 PM, Jiong Wang <jiwang at tilera.com> wrote: >> >> Hi Chris, >>> >>> could you please comment on committing TILE-Gx backend into community? >>> >> Hi Jiong, >> >> I don't have any special advice here. It sounds like the general >> functionality level is high enough. Taking it into mainline sounds great, >> so long as it is reviewed by someone. >> > > thanks. > > I will commit after rebasing & re-testing the code on latest llvm > mainline. And I will follow http://llvm.org/docs/**HowToAddABuilder.html<http://llvm.org/docs/HowToAddABuilder.html>to setup a TILE-Gx buildbot. >No, read what he said again: so long as it is reviewed by someone. That hasn't happened yet, and you need it to happen before you commit. As to how you can get the entire backend reviewed, I gave advice on that front in my initial response to your initial email, the very first response you got: Unfortunately, this is a *huge* amount of code, and so it is likely to take> quite some time to really get reviews on all of it. Please be patient in > that regard.The best thing you can do to help accelerate the process is to dive in and> contribute to the shared parts of the open source project to help the > current maintainers, and maybe even free up some of their time to review > your patches. Because LLVM is open source, it is really important to the > project for folks working on a specific target, backend, or application to > also help share the ongoing maintenance and improvement costs of the core > shared infrastructure in the compiler.-Chandler -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20130324/8907cac6/attachment.html>
Maybe Matching Threads
- [LLVMdev] About commit TILE-Gx backend to community repository and default disabled
- [LLVMdev] About commit TILE-Gx backend to community repository and default disabled
- [LLVMdev] About commit TILE-Gx backend to community repository and default disabled
- [LLVMdev] About commit TILE-Gx backend to community repository and default disabled
- [LLVMdev] About commit TILE-Gx backend to community repository and default disabled