<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 13, 2014 at 5:03 PM, Jiangning Liu <span dir="ltr"><<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi Chandler,<div><br></div><div>Attached patch is fix this issue. OK to commit?</div>
</div></blockquote><div><br></div><div>Mostly...<br></div><div><br></div><div>I don't really understand the changes in test/CodeGen/ARM/global-merge-1.ll though. You seem to be removing a bunch of tests (perhaps in addition to merging in the test from test/Transforms/GlobalMerge/ARM/...) which doesn't make sense to me. </div>
<div><br></div><div>Otherwise everything seems good.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div><br></div><div>BTW, should we also have a regression test to check the libraries' dependency?</div></div></blockquote><div><br></div><div>Not sure of an easy way to do this, but maybe its worthwhile.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div> I didn't expect this is blocking others' build. Sorry about that!</div>
</div></blockquote><div><br></div><div>It's fine, it didn't break any build bots, etc. Just some uses of LLVM that assumed the scalar optimization library didn't have these dependencies were broken.</div></div>
</div></div>