Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

[WIP] Use target_ssize_t for Lowering::CastInfo::typeMin typeMax typeMask#19545

Closed
echesakov wants to merge 1 commit into
dotnet:masterfrom
echesakov:UseTargetSizeTForLoweringCastInfo
Closed

[WIP] Use target_ssize_t for Lowering::CastInfo::typeMin typeMax typeMask#19545
echesakov wants to merge 1 commit into
dotnet:masterfrom
echesakov:UseTargetSizeTForLoweringCastInfo

Conversation

@echesakov
Copy link
Copy Markdown

No description provided.

@echesakov echesakov added the * NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons) label Aug 18, 2018
@mikedn
Copy link
Copy Markdown

mikedn commented Aug 18, 2018

FYI #19296 removes CastInto and the new code specifically avoids using size_t in problematic cases. I hope to finish that by the end of next week.

@mikedn
Copy link
Copy Markdown

mikedn commented Aug 24, 2018

#19296 is done now. If someone has time to review such a change it's another matter :)

@echesakov
Copy link
Copy Markdown
Author

@mikedn Thanks for heads up!

@echesakov echesakov closed this Sep 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

* NO MERGE * The PR is not ready for merge yet (see discussion for detailed reasons)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants