search for: d64123

...>> positive comments on the review thread. Thank you for driving this forward! >> >> -Chris >> >> >> On Jul 4, 2019, at 9:50 PM, Rui Ueyama <ruiu at google.com> wrote: >> >> Hi all, >> >> I wrote a tool <https://reviews.llvm.org/D64123> to batch-rename >> variable names so that they are in camelCase, and I applied the tool to lld >> subdirectory. You can see my change at https://reviews.llvm.org/D64121. >> If you have any comments, please reply. >> >> If people are happy about this change, I can d...

...ooks really great to me Rui, and I’m also pleased to see the positive comments on the review thread. Thank you for driving this forward! -Chris > On Jul 4, 2019, at 9:50 PM, Rui Ueyama <ruiu at google.com> wrote: > > Hi all, > > I wrote a tool <https://reviews.llvm.org/D64123> to batch-rename variable names so that they are in camelCase, and I applied the tool to lld subdirectory. You can see my change at https://reviews.llvm.org/D64121 <https://reviews.llvm.org/D64121>. If you have any comments, please reply. > > If people are happy about this change, I...

...eview thread. Thank you for driving this forward! >>> >>> -Chris >>> >>> >>> On Jul 4, 2019, at 9:50 PM, Rui Ueyama <ruiu at google.com> wrote: >>> >>> Hi all, >>> >>> I wrote a tool <https://reviews.llvm.org/D64123> to batch-rename >>> variable names so that they are in camelCase, and I applied the tool to lld >>> subdirectory. You can see my change at https://reviews.llvm.org/D64121. >>> If you have any comments, please reply. >>> >>> If people are happy about...

On Mon, Jun 10, 2019 at 6:27 PM Michael Platings <Michael.Platings at arm.com> wrote: > Hi Rui, > > > > As per the provisional plan [1] we’re still at step 1: improving git > blame. The status of this is that there are some fairly mature patches in > the Git project’s queue [2], and I’m hopeful that it will be accepted in > something close to its current form. >

...>>>>> -Chris >>>>> >>>>> >>>>> On Jul 4, 2019, at 9:50 PM, Rui Ueyama <ruiu at google.com> wrote: >>>>> >>>>> Hi all, >>>>> >>>>> I wrote a tool <https://reviews.llvm.org/D64123> to batch-rename >>>>> variable names so that they are in camelCase, and I applied the tool to lld >>>>> subdirectory. You can see my change at https://reviews.llvm.org/D64121. >>>>> If you have any comments, please reply. >>>>> >&gt...

Hi Rui, I’m totally positive on switching to camelCase convention. In fact I have been always uncomfortable with the current naming approach. My only suggestion/concern is that we should provide a transition path not only for the trunk code in the repository, but for eventual out-of-trunk code with implementations of custom architectures. I have currently a custom backend implemented on top of

...ists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev > > > Links: > ------ > [1] https://reviews.llvm.org/D64123 > [2] https://llvm.org/docs/Proposals/VariableNames.html#provisional-plan > [3] > https://public-inbox.org/git/20190515214503.77162-8-brho at google.com/T/ > [4] https://github.com/tensorflow/mlir > [5] > https://github.com/tensorflow/mlir/blob/master/lib/Transforms/LoopUnrollAnd...

...-dev > > _______________________________________________ > > LLVM Developers mailing list > > llvm-dev at lists.llvm.org > > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev > > > > > > Links: > > ------ > > [1] https://reviews.llvm.org/D64123 > > [2] https://llvm.org/docs/Proposals/VariableNames.html#provisional-plan > > [3] > > https://public-inbox.org/git/20190515214503.77162-8-brho at google.com/T/ > > [4] https://github.com/tensorflow/mlir > > [5] > > > https://github.com/tensorflow/mlir/blob/m...