<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Nov 15, 2019 at 10:58 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Since you are going to rewrite the mlir history anyway, you can<br>
probably delete accidentally checked in large files if any.<br></blockquote><div><br></div><div>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">https://github.com/joker-eph/llvm-project-with-mlir/blob/master/mlir/g3doc/includes/img/view-operation.svg</a> (155kB) </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<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></blockquote><div><br></div><div>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. </div><div>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.</div><div><br></div><div>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?</div><div><br></div><div>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">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></div><div><br></div><div>-- </div><div>Mehdi</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
On Fri, Nov 15, 2019 at 2:03 AM Mehdi AMINI via llvm-dev<br>
<<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> (bcc: <a href="mailto:mlir@tensorflow.org" target="_blank">mlir@tensorflow.org</a> FYI)<br>
><br>
> I am following-up on the integration of MLIR in LLVM as a subproject (Re: <a href="http://lists.llvm.org/pipermail/llvm-dev/2019-October/135687.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2019-October/135687.html</a> ).<br>
><br>
> We're aiming to integrate into the monorepo next month. Right now our intent is for MLIR to live in a top-level directory in parallel to clang, lldb, lld, etc.<br>
> Our top option for the integration is to perform a git subtree merge to bring the MLIR history into the monorepo, here is a prototype: <a href="https://github.com/joker-eph/llvm-project-with-mlir" rel="noreferrer" target="_blank">https://github.com/joker-eph/llvm-project-with-mlir</a><br>
><br>
> If you're curious to try it, at the moment it needs a specific CMake invocation:<br>
><br>
> cmake -G Ninja ../llvm/ -DLLVM_TARGETS_TO_BUILD="host" -DLLVM_EXTERNAL_PROJECTS=mlir -DLLVM_EXTERNAL_MLIR_SOURCE_DIR={path to repo}/mlir/<br>
><br>
> We'll hook into -DLLVM_ENABLE_PROJECTS after landing.<br>
><br>
> Let me know if you have any comment about this!<br>
><br>
> Cheers,<br>
><br>
> --<br>
> Mehdi<br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
<br>
<br>
<br>
-- <br>
宋方睿<br>
</blockquote></div></div></div></div>