<div dir="auto"><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;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>