Skip to content

added default virtual dtor to 'computation' and missing return … - #311

Open
ct-clmsn wants to merge 16 commits into
Tiramisu-Compiler:masterfrom
ct-clmsn:physl_2019
Open

added default virtual dtor to 'computation' and missing return …#311
ct-clmsn wants to merge 16 commits into
Tiramisu-Compiler:masterfrom
ct-clmsn:physl_2019

Conversation

@ct-clmsn

@ct-clmsn ct-clmsn commented Dec 29, 2019

Copy link
Copy Markdown

clang-8 identifies two compilation issues. the first is a missing virtual dtor in computation, the second is a missing return value in auto_scheduler::create_initial_computation_graph.

…object to 'auto_scheduler::create_initial_computation_graph'
@ct-clmsn

Copy link
Copy Markdown
Author

fixes for #312 and #313

@ct-clmsn ct-clmsn changed the title added default virtual destructor to 'computation' and missing return … added default virtual dtor to 'computation' and missing return … Dec 29, 2019
@ct-clmsn

ct-clmsn commented Dec 30, 2019

Copy link
Copy Markdown
Author

build error is related to an llvm versioning issue? there's an undefined reference in the halide portion of the compilation.

…t from 'function' this improves the extensibility of the code generator to new languages
@ct-clmsn

ct-clmsn commented Jan 3, 2020

Copy link
Copy Markdown
Author

added a modification to where 'protected' is declared over methods in 'class function'. the methods that are now declared protected enable the 'function' class to be extensible. making the class 'function' extensible makes tiramisu capable of supporting new languages (ie: physl).

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.

1 participant