<div dir="ltr"><div style>LGTM (modulo one comment below). Please submit this if it works for your configuration where compiler-rt is checked out in different place.</div><div style>Thanks!</div><div><br></div><div>-compiler_rt_lit_cfg = os.path.join(llvm_src_root, "projects", "compiler-rt",</div>
<div>-                                   "lib", "lit.common.cfg")</div><div>+compiler_rt_src_root = getattr(config, 'compiler_rt_src_root', None)</div><div>+compiler_rt_lit_cfg = os.path.join(compiler_rt_src_root, "lib",</div>
<div>+                                   "lit.common.cfg")</div><div>You would need get_required_attr() here (as in other .cfg files) or check that attr value is not None<br></div><div style><br></div><div style>
<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 24, 2013 at 3:42 AM, Greg Fitzgerald <span dir="ltr"><<a href="mailto:gregf@codeaurora.org" target="_blank">gregf@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal">Attached is a patch to fix the compiler-rt build when it is in a directory outside “llvm/projects”.  Can you please review?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Thanks,<u></u><u></u></p><p class="MsoNormal">Greg<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#222222;background:white">Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted</span><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#222222"><br>
<span style="background:white">by The Linux Foundation</span></span><u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>

</div>