[SYCL][WIP] Reduce compile time builtin#21688
Draft
koparasy wants to merge 2 commits intointel:syclfrom
Draft
Conversation
…lper customization points so <sycl/builtins.hpp> no longer needs to include the full <sycl/multi_ptr.hpp> definition. Preserve multi_ptr builtin behavior once multi_ptr.hpp is included, and update include-deps / regression coverage. On a TU including only <sycl/builtins.hpp>: Host-only Frontend: 601.943 ms -> 572.657 ms (-4.87%) Device-only Frontend: 592.140 ms -> 457.361 ms (-22.76%) multi_ptr.hpp parse time dropped from 18.708 ms -> 0 ms on host multi_ptr.hpp parse time dropped from 108.662 ms -> 0 ms on device
…uiltins.hpp`. This keeps the relational builtins path self-contained by introducing a narrow relational_mask_widen helper for relation-mask widening, and replaces the old transitive dependencies with the direct headers that builtins actually needs (generic_type_traits.hpp, half_type.hpp). It also adds the missing direct exception.hpp include for ext/oneapi/bf16_storage_builtins.hpp and updates the affected include-deps / IR checks. Measured with measure_builtin.cpp and -ftime-trace against the base branch: host Total ExecuteCompiler: 604.853 ms -> 533.452 ms (-71.401 ms, -11.80%) host Total Frontend: 600.196 ms -> 529.366 ms (-70.830 ms, -11.80%) device Total ExecuteCompiler: 598.557 ms -> 420.885 ms (-177.672 ms, -29.68%) device Total Frontend: 593.798 ms -> 416.060 ms (-177.738 ms, -29.93%) changes
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.