<div dir="ltr"><div dir="ltr">Hi Jon,<div><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">Libomptarget has broken free of copy&paste of headers by declaring a build time requirement for the llvm source tree, from which it includes otherwise standalone headers.</blockquote><div><br></div><div>Thanks very much for pointing this out. It seems reasonable to assume the presence of an LLVM source tree -- several compiler-rt tests rely on LLVM tools already. I'll double check with the compiler-rt maintainers, but this sounds like a good solution to me.</div><div><br></div><div>-- Lang. </div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 25, 2021 at 2:21 AM Jon Chesterfield via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</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"><div dir="auto"><div><div class="gmail_quote"><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>
   2. Re: Inclusion of the ORC runtime in compiler-rt.<br> <br><br><span style="font-family:sans-serif;font-size:12.8px">> InstrProfData.inc is shared between LLVM and compiler-rt:</span><br style="font-family:sans-serif;font-size:12.8px"><span style="font-family:sans-serif;font-size:12.8px">></span><br style="font-family:sans-serif;font-size:12.8px"><span style="font-family:sans-serif;font-size:12.8px">></span><br style="font-family:sans-serif;font-size:12.8px"><br style="font-family:sans-serif;font-size:12.8px"><br style="font-family:sans-serif;font-size:12.8px"><span style="font-family:sans-serif;font-size:12.8px">There's no special mechanism to keep those in sync, we manage them manually.</span><br style="font-family:sans-serif;font-size:12.8px"><br style="font-family:sans-serif;font-size:12.8px"><br style="font-family:sans-serif;font-size:12.8px"><span style="font-family:sans-serif;font-size:12.8px">Thanks Petr. That's about the same amount of code that ORC will need to</span><br style="font-family:sans-serif;font-size:12.8px"><span style="font-family:sans-serif;font-size:12.8px">share I think. Glad there's a precedent to follow.</span></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Libomptarget has broken free of copy&paste of headers by declaring a build time requirement for the llvm source tree, from which it includes otherwise standalone headers.</div><div dir="auto"><br></div><div dir="auto">This is much nicer than manually updating, or having tests that do the diff, or build scripts that do the copy.</div><div dir="auto"><br></div><div dir="auto">If orc doesn't have a cmake variable from which it can find llvm, it can warn about that and decline to build.</div><div dir="auto"><br></div><div dir="auto">This will break people who build orc without llvm available anywhere on disk, until they download llvm or build a local skeleton directory containing copies of the files. That may be an empty set of people. Noone complained for libomptarget.</div><div dir="auto"><br></div><div dir="auto">Please seriously consider this strategy instead of duplicating the files.</div><div dir="auto"><br></div><div dir="auto">Jon</div></div>
_______________________________________________<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>
</blockquote></div>