Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Modularize <type_traits> #335

Merged
merged 140 commits into from
Dec 19, 2022
Merged

Modularize <type_traits> #335

merged 140 commits into from
Dec 19, 2022

Conversation

miscco
Copy link
Collaborator

@miscco miscco commented Nov 30, 2022

This goes ahead and rips type_traits apart. We will need this eventually for ranges, as otherwise the required changes become a mess

@miscco miscco requested review from griwes and wmaxey November 30, 2022 11:28
@miscco miscco force-pushed the modularize_type_traits branch 3 times, most recently from 77514de to 21e8f82 Compare December 6, 2022 16:56
Copy link
Collaborator

@griwes griwes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks really good; I do really like how many of the fundamental but non-trivial traits have gotten themselves some compiler intrinsics in the meantime. Just a few comments below.

@miscco miscco added the testing: internal ci in progress Currently testing on internal NVIDIA CI (DVS). label Dec 8, 2022
@miscco miscco force-pushed the modularize_type_traits branch 2 times, most recently from 98b03c2 to de07bf9 Compare December 15, 2022 12:05
@miscco miscco force-pushed the modularize_type_traits branch from a639f39 to 615cbf8 Compare December 16, 2022 07:45
we missed some annotations and NODEBUG is a nice trait...

fixed all include handlings
@miscco miscco force-pushed the modularize_type_traits branch from 615cbf8 to 4c38345 Compare December 16, 2022 07:50
@miscco miscco added testing: internal ci passed Passed internal NVIDIA CI (DVS). and removed testing: internal ci in progress Currently testing on internal NVIDIA CI (DVS). labels Dec 19, 2022
@miscco miscco merged commit 5a9319c into NVIDIA:main Dec 19, 2022
@miscco miscco deleted the modularize_type_traits branch December 19, 2022 09:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing: internal ci passed Passed internal NVIDIA CI (DVS).
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants