<div dir="ltr"><div class="gmail_default" style="font-family:monospace;font-size:small;color:#000000">Adding a test to make sure they are in sync will be useful.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 14, 2020 at 1:12 AM Pavel Kosov via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">kpdev42 added a comment.<br>
<br>
In D86890#2270337 <<a href="https://reviews.llvm.org/D86890#2270337" rel="noreferrer" target="_blank">https://reviews.llvm.org/D86890#2270337</a>>, @filcab wrote:<br>
<br>
> I would love to have a single source of truth, but this basically makes it<br>
> impossible to just build compiler-rt by itself, no?<br>
><br>
> That path is also “very non portable”, given that our cmake build system<br>
> can find/be directed to the source for the projects in different places.<br>
><br>
> Thank you,<br>
> Filipe<br>
<br>
Thank you for the feedback.<br>
If it is a important case when compiler-rt should be build separately from llvm-project, <br>
then maybe I'll add a motivation for it to file description (like `compiler-rt should have a possibility to be built out of tree`)<br>
and add a test to compiler-rt which will check these files to be identical?<br>
<br>
Or it is better to leave everything as it is? :)<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D86890/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D86890/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D86890" rel="noreferrer" target="_blank">https://reviews.llvm.org/D86890</a><br>
<br>
</blockquote></div>