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

Compilation bug preventing divide and conquer loop parallelization #38

Closed
timkaler opened this issue Jan 30, 2018 · 1 comment
Closed

Comments

@timkaler
Copy link

To reproduce: https://github.mit.edu/Supertech-Connectomics/sift_features/tree/1-24-2018-tapir-bug (branch 1-24-2018-tapir-bug) and checkout commit 95540f89caf784d42f6b8ca4614efd10328edbf3 on that branch.

A workaround that appears to avoid the compilation problem is to change the return value of Section::parameter_optimization to be void (and return result via argument reference).

neboat added a commit that referenced this issue Feb 8, 2018
neboat added a commit that referenced this issue Feb 8, 2018
@neboat
Copy link
Collaborator

neboat commented Feb 9, 2018

I've tested the committed fix on the source code specified in this issue, and I now observe no errors in compilation.

@neboat neboat closed this as completed Feb 9, 2018
neboat added a commit that referenced this issue Feb 16, 2018
neboat added a commit that referenced this issue Mar 24, 2018
wsmoses pushed a commit that referenced this issue Nov 1, 2018
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue Dec 19, 2018
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue Dec 20, 2018
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue Dec 20, 2018
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue Jan 12, 2019
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue Mar 14, 2019
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue May 9, 2019
stelleg pushed a commit to stelleg/Tapir-LLVM that referenced this issue May 17, 2019
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

No branches or pull requests

2 participants