<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Mar 30, 2009, at 11:36 AM, Misha Brukman wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); margin-top: 0pt; margin-right: 0pt; margin-bottom: 0pt; margin-left: 0.8ex; padding-left: 1ex; "><div class="im">>> @@ -1,4 +1,4 @@<br>>> -; RUN: llvm-as < %s | llc -march=ppc64 -debug |& not grep store<br>>> +; RUN: llvm-as < %s | llc -march=ppc64 -debug |& not grep {= store}<br>><br>> I thought that "store" instructions were never assigned to a<br>> variable...<br><br></div>This test is grepping through the -debug output of llc, and the stores<br>its<br>looking for do happen to be printed with =.<br><br>Grepping through -debug output isn't very nice in any case, but at the<br>moment there aren't any better ways to express this specific test.<font color="#888888"><br></font></blockquote></div><br>You can write a unittest to check this.  It might require some refactoring to be able to run all of LLC's passes separately from running the tool on the command line, since they are currently hard-coded in its main() function.<br></span></blockquote></div><br><div>I don't think we want unit tests for passes.  We only want unit tests for low level APIs.</div><div><br></div><div>-Chris</div></body></html>