rdaly525 / coreir

BSD 3-Clause "New" or "Revised" License
100 stars 24 forks source link

Fix Simulator Plugin in Interpreter for Running H2H Simulation #873

Closed joyliu37 closed 4 years ago

joyliu37 commented 4 years ago

I make a small changes in interpreter.cpp to make memory rewrite could be simulated.

rsetaluri commented 4 years ago

@joyliu37 can you clean this PR up and reduce it to just the things relevant to your change? It looks like a lot of extraneous things in here; including effects of a merge. you should rebase on master too.

rdaly525 commented 4 years ago

@joyliu37, agreed with @rsetaluri, could you do the master rebase?

joyliu37 commented 4 years ago

Should I do git rebase master instead of git merge master. Sorry about that, I have poor git knowledge.

rdaly525 commented 4 years ago

rebase is preferable, but sometimes that is too ridiculous, so I often find myself doing the following:

>git checkout master
>git pull
>git checkout <my_branch>
>git merge master
>git reset master
>git add <your changes>
>git commit
>git push -f

Basically this gets your branch up to date with master and gives it a clean commit history. The local changes should still be there, but requires you to re add/commit.

rsetaluri commented 4 years ago

@joyliu37 see #875. please work off that if it is appropriate. In this case I think a rebase is best.

Also, your editor seems to be automatically cleaning up whitespace -- please do not commit the changes caused by that. Make sure whatever you add and commit is actually what is relevant for your change. I recommend against git commit -a.

joyliu37 commented 4 years ago

I also found the whitespace substitute issue. I will check your commit.

rsetaluri commented 4 years ago

Closing in favor of #875