I've committed this as r105796. We can factor these out to tablegen files if there is a desire to do so in a subsequent patch.<div><br><div class="gmail_quote">On Thu, Jun 10, 2010 at 10:18 AM, Zhanyong Wan (λx.x x) <span dir="ltr"><<a href="mailto:wan@google.com">wan@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi,<br>
<br>
I uploaded a patch to <a href="http://codereview.appspot.com/1653043" target="_blank">http://codereview.appspot.com/1653043</a> to avoid<br>
repeating the lists of operators twice in RecursiveASTVisitor.h.  This<br>
should make the code easier to maintain, but we no longer need to<br>
worry about the duplicates going out of sync.  What do you think?<br>
<br>
Chandler and I discussed the option of generating the list from a .td<br>
file.  I'm fine with that idea, but prefer to wait for the list to be<br>
needed at another place before doing that.  Thoughts?  Thanks,<br>
<font color="#888888"><br>
--<br>
Zhanyong<br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</font></blockquote></div><br></div>