-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Finish support for list-of-targets #11382
Merged
Merged
Commits on May 22, 2022
-
Finish support for list-of-targets
This finishes the work started in apache#11173 to support 'external codegen' targets in the N build-like API surfaces. - It turns out it's ok if a build is given only a single 'external codegen' target, so remove that check in CompilationConfig::Init. When Collage builds a 'candidate partition' it does so for a single target. As far as Collage is concerned it does not care whether the target is regular (eg Target("cuda")), or for a specific external codegen (eg Target("cutlass")), it just passes the target into the build. - Add CompilationConfig::FindPrimitiveTargetForKind which I'll later need to retrieve the external codegen Target instance corresponding to a "Compiler" attribute value. - Target.update_target_host_consist was supporting three API styles: - single target - map from device type to target - map from target to IRModule (for the ir_to_runtime API) I replaced all those calls with a more specialized 'canonicalize' call: - Target.canonicalize_target_and_host - Target.canonicalize_multi_targets_and_host - Target.canonicalize_target_map_and_host In particular, all the tuning interfaces (task extraction, tuning, tuning records) all explicitly *do not* support multiple targets since the underlying code just doesn't support that.
Configuration menu - View commit details
-
Copy full SHA for 5dfaf07 - Browse repository at this point
Copy the full SHA 5dfaf07View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5f8dce6 - Browse repository at this point
Copy the full SHA 5f8dce6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 995778e - Browse repository at this point
Copy the full SHA 995778eView commit details -
- Fix model_library_format handling of target.
- Improve comments in compilation_config.h
Configuration menu - View commit details
-
Copy full SHA for ea0ac06 - Browse repository at this point
Copy the full SHA ea0ac06View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5f11a25 - Browse repository at this point
Copy the full SHA 5f11a25View commit details -
- Fix micro library format tests
- Rev micro library format from 5 to 6 - Use Target.current() in a few places
Configuration menu - View commit details
-
Copy full SHA for 81a75a2 - Browse repository at this point
Copy the full SHA 81a75a2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 31a1585 - Browse repository at this point
Copy the full SHA 31a1585View commit details -
- Woops, one more device: target map left
- Handle host already being in Target
Configuration menu - View commit details
-
Copy full SHA for c9687a5 - Browse repository at this point
Copy the full SHA c9687a5View commit details -
Configuration menu - View commit details
-
Copy full SHA for de2ffd1 - Browse repository at this point
Copy the full SHA de2ffd1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1843254 - Browse repository at this point
Copy the full SHA 1843254View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5291e5b - Browse repository at this point
Copy the full SHA 5291e5bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 715daab - Browse repository at this point
Copy the full SHA 715daabView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.