Skip to content
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

Dev #3

Closed
wants to merge 3 commits into from
Closed

Dev #3

wants to merge 3 commits into from

Conversation

qqldd
Copy link
Contributor

@qqldd qqldd commented Oct 6, 2014

No description provided.

1. ptx_sim.h::473, stack cannot use incomplete type "operand_info" which is a forward declaration. The reason is underlying implementation of stack is deque which need a complete type. It is better to remove forward declaration by break circular dependence for future fixes. It is also benefit unit test
2. shader.h::1334, this hack cannot pass clang. Clang does not allow a array with not a explicit size. Please fix this hack by correct implementation as soon as possible
3. The default parameter causes clang to fail because it frustrate the compiler. This is still under discussion whether such implementation is correct. http://stackoverflow.com/questions/18313509/default-argument-gcc-vs-clang. I changed it to two constructors to avoid confusion.
…g and dependency file too big without a make clean
@qqldd qqldd closed this Oct 6, 2014
@qqldd qqldd deleted the dev branch October 6, 2014 22:59
@qqldd qqldd restored the dev branch October 6, 2014 23:00
deval281shah referenced this pull request in deval281shah/gpgpu-sim_distribution Nov 9, 2018
Support for CDP and minor bug fixes
aamodt pushed a commit that referenced this pull request Feb 15, 2019
aamodt pushed a commit that referenced this pull request Apr 19, 2019
tgrogers pushed a commit to tgrogers/gpgpu-sim_distribution that referenced this pull request Jun 4, 2019
VijayKandiah referenced this pull request in VijayKandiah/gpgpu-sim_distribution Feb 16, 2021
Fix for bugs in lazy write handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants