<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 12, 2010, at 3:51 PM, Bob Wilson wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div><br>On Oct 12, 2010, at 3:47 PM, Jim Grosbach wrote:<br><br><blockquote type="cite">Not really. It might make sense to put some of those bits on the same line, though, which would help (the portions that are encoding different pieces of the same operand, e.g.). Something like:<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite"> bits<5> Dd; bits<5> Dn; bits<5> Dm;<br></blockquote><blockquote type="cite"> let Inst{3-0} = Dm{3-0}; let Inst{5} = Dm{4};<br></blockquote><blockquote type="cite"> let Inst{19-16} = Dn{3-0}; let Inst{7} = Dn{4};<br></blockquote><blockquote type="cite"> let Inst{15-12} = Dd{3-0}; let Inst{22} = Dd{4};<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">We may also be able to later refactor some of this sort of thing to put lots of stuff in the base classes rather than the per-instruction patterns. That would help even more. I'm not sure our definitions are laid out in a way that makes that very easy, though (dunno that they're not, either).<br></blockquote><br>Can we try to put it in the base classes? It would be worth the effort to do some refactoring to make that possible. I think it will be less work to do it that way from the start rather than doing it this way first and then refactoring it all later.<br></div></blockquote><div><br></div>I agree.</div><div><br></div><div>Evan</div><div><br><blockquote type="cite"><div>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br></div></blockquote></div><br></body></html>