<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 22, 2016 at 8:59 AM, bryant <span dir="ltr"><<a href="mailto:3.14472+reviews.llvm.org@gmail.com" target="_blank">3.14472+reviews.llvm.org@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">bryant added a comment.<br>
<span class=""><br>
> Actually, can you please add a test for this before you commit?<br>
> You can use -print-memoryssa -analyze to get it to dump memoryssa.<br>
> If you need specific API calls, please add to the unit tests for MemorySSA.<br>
<br>
</span>I'm having trouble getting `-analyze` to work:<br>
<br>
  16:58:24 ~/3rd/llvm/test> opt -print-memoryssa -analyze < Transforms/Util/MemorySSA/<wbr>precision-call-clobbers.ll<br>
  ... # stuff<br>
  Printing analysis 'Memory SSA Printer' for function 'f':<br>
  Pass::print not implemented for pass: 'Memory SSA Printer'!</blockquote><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If the goal is to run FileCheck on the MemorySSA debug output, shouldn't -print-memoryssa alone be sufficient?<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div>It is now.</div><div>It did not used to be because MemorySSA was lazy, and when it was lazy, -analyze was required to instantiate the result.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D26659" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D26659</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>