<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Nov 15, 2019 at 11:43 AM Fāng-ruì Sòng <<a href="mailto:maskray@google.com">maskray@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Nov 15, 2019 at 11:15 AM Mehdi AMINI <<a href="mailto:joker.eph@gmail.com" target="_blank">joker.eph@gmail.com</a>> wrote:<br>
><br>
> On Fri, Nov 15, 2019 at 10:58 AM Fāng-ruì Sòng <<a href="mailto:maskray@google.com" target="_blank">maskray@google.com</a>> wrote:<br>
>><br>
>> Since you are going to rewrite the mlir history anyway, you can<br>
>> probably delete accidentally checked in large files if any.<br>
><br>
><br>
> Good point, I checked and this is the largest file in the history of the repo as far as I can tell: <a href="https://github.com/joker-eph/llvm-project-with-mlir/blob/master/mlir/g3doc/includes/img/view-operation.svg" rel="noreferrer" target="_blank">https://github.com/joker-eph/llvm-project-with-mlir/blob/master/mlir/g3doc/includes/img/view-operation.svg</a> (155kB)<br>
>><br>
>><br>
>> * I don't know whether the file CONTRIBUTING.md is still appropriate,<br>
>> at least for the Code of Conduct, LLVM has its own version.<br>
>> * g3doc/ seems a very Google specific name. Does `docs/` work?<br>
>> * bindings/python/pybind.cpp - does it have to be an in-tree plugin?<br>
>> * The Apache 2 license headers are verbose. LLVM uses<br>
>> SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception.<br>
><br>
><br>
> Absolutely! All of these points (except the python bindings) are on my TODO list of things to do at the time of the merge. At the moment there is just a script continuously performing the merge in my test repository so this is an exact view of the current state of the public repo.<br>
> I could also try to rewrite the license in the header in all the history of the repository, but I'm not sure it won't be brittle in practice, I was planning to do an update to all the files before pushing.<br>
><br>
> I'll send the final version of the repo next month, I can CC you if you'd like to review this before we push it?<br>
<br>
Sure!<br>
<br>
> For the python bindings, these are intended to provide some equivalent facility to the LLVM python bindings: <a href="https://github.com/joker-eph/llvm-project-with-mlir/tree/master/llvm/bindings/python" rel="noreferrer" target="_blank">https://github.com/joker-eph/llvm-project-with-mlir/tree/master/llvm/bindings/python</a> ; while they need some work at the moment, I think we will want to have bindings though.<br>
<br>
<br>
Another thing is that after you make CMake work<br>
-DLLVM_ENABLE_PROJECTS='...;mlir;...', it'd be nice to reorder that<br>
commit to the very beginning of the MLIR history. People want to<br>
bisect and build at any commit in the pre-monorepo history. If they<br>
can only build at the last commit, that will still be very<br>
inconvenient.</blockquote><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">I don’t think it is possible: it requires changes in LLVM itself but also the revision pre-merge contain *only* MLIR and not LLVM: these revision won’t build as is.</div><div dir="auto">I think bisection in the pre-merge history will have to use “commit date” to find a compatible revision in LLVM, and at this point the need for the more verbose syntax I send earlier isn’t the more annoying part.</div><div dir="auto"><br></div><div dir="auto">Hopefully going back far in time is not something that will be common.</div><div dir="auto"><br></div><div dir="auto">— </div><div dir="auto">Mehdi</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div>