-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AEx: Automated processor exploration #235
base: main
Are you sure you want to change the base?
Conversation
Add the methodology from AEx JSPS paper. Simultaneously upstream other small explorer changes and fix up tests relevant to those.
@@ -8,6 +8,13 @@ Notable changes and features | |||
Now it's possible to optionally give a list of pairs of | |||
<address-space-name>,<data-start>. | |||
- Added the same option to generatebits with the same syntax. | |||
- Automated Processor Exploration | |||
- Open-sourcing the methodology described in: | |||
"AEx: Automated High-Level Synthesis of Compiler Programmable Co-Processors" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls add a link to the paper.
|
||
BusMergeMinimizer tries to optimize a machine's interconnection network by "merging" two buses into one, which then has the connections of both original buses. | ||
A simulation trace based heuristic decides which buses to merge: the idea is that when buses are seldom active at the same time, they can be merged without hurting performance much. | ||
There is more information in the paper "Greedy Heuristics for Transport Triggered Architecture Interconnect Exploration". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls add a link here as well.
@@ -0,0 +1,361 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGPL v2.1 license should be used here.
@@ -0,0 +1,670 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGPL v2.1 (also to other source files, not mentioning them separately).
using namespace TTAMachine; | ||
using namespace HDB; | ||
|
||
const int ALLOPERATIONAMCHINE = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change to enum?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also the names should be underscore separated.
@@ -30,32 +30,36 @@ | |||
* @note rating: red | |||
*/ | |||
|
|||
#include <vector> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file also seems to have the MIT license.
Let's pull this in for 2.2? |
Add the methodology from AEx JSPS paper.
Simultaneously upstream other small explorer changes and fix up tests relevant to those.