| --- Log opened Thu May 02 00:00:40 2013 | ||
| stekern | juliusb: you better agree with me about moving decode signal registering into a seperate module now, because I've already done the work ;) | 06:48 |
|---|---|---|
| stekern | https://github.com/skristiansson/mor1kx/commit/9194bab37056b8d0a153093442d2396be2aec057 | 06:48 |
| stekern | it's kind of large, but it's really no functional change, just restructuring things | 06:56 |
| @juliusb | stekern: haha, OK, just reviewing the patch | 17:21 |
| stekern | juliusb: thanks ;) | 17:27 |
| @juliusb | what parts of the new decode_execute_capp are cappuccino-specific? | 17:28 |
| stekern | basically the branch resolving and the interlock stuff (i.e. decode bubble) | 17:29 |
| @juliusb | ah OK | 17:29 |
| @juliusb | I like the renaming of all outputs to be prefixed with decode_ | 17:29 |
| stekern | although, I guess they could be of interest for another pipeline implementation as well | 17:30 |
| @juliusb | I'm in 2 minds about the naming of the block as decode_execute_cappuccino, I'm wondering if we should keep the naming more generic, but it really doesn't matter too much | 17:30 |
| stekern | mmm, that makes it 4 minds, I was thinking the same, hence that comment in the commit message | 17:31 |
| @juliusb | yeah, i say just leave the patch as is | 17:34 |
| stekern | I decided on leaving it like that, since then it sticks out as something that is not used across all pipelines, making it more explicit that espresso doesn't use it | 17:37 |
| @juliusb | yep | 17:37 |
| @juliusb | fair enough | 17:39 |
| stekern | I take that as you're happy enough with it? | 18:20 |
| * stekern is flipping away the safety glass from the big red 'COMMIT' button | 18:22 | |
| --- Log closed Fri May 03 00:00:42 2013 | ||
Generated by irclog2html.py 2.15.2 by Marius Gedminas - find it at mg.pov.lt!