<div dir="ltr">Hi Chandler,<div class="gmail_extra"><br><div class="gmail_quote">2014-06-14 0:11 GMT+08:00 Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank">chandlerc@google.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">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><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></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div>
<div>Otherwise everything seems good.</div><div class=""><div><br></div></div></div></div></div></blockquote><div><br></div><div>I kept the original version of test/CodeGen/ARM/global-merge-1.ll.</div><div><br></div><div>
I committed as r210951 and r210952. r210952 is to completely remove folder test/Transforms/GlobalMerge.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class=""><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><br></div><div>BTW, should we also have a regression test to check the libraries' dependency?</div></div></blockquote><div><br></div></div><div>Not sure of an easy way to do this, but maybe its worthwhile.</div>
</div></div></div></blockquote><div><br></div><div>I really didn't realize this small change of add macro INITIALIZE_TM_PASS raised such a lot of nervous. Sorry for any inconvenience caused! It seems at least Qunentin and Rafael also thought we could make this kind of change at first look. Hopefully I'm the last one making this kind of mistake.</div>
<div><br></div><div>So I think it's worthwhile to add this kind of regression test. At least it can enhance llvm infrastructure robustness and clear layers on libxxx.a/so support.</div><div> </div><div>Thanks,</div><div>
-Jiangning</div><div><br></div></div></div></div>