Skip to content

Conversation

@guitargeek
Copy link
Contributor

@guitargeek guitargeek commented Mar 25, 2025

Use GetSizeOfType from CppInterOp.

@guitargeek guitargeek force-pushed the TClingTypeInfo_Size branch from 1abb372 to 7a253dd Compare March 25, 2025 15:22
@guitargeek guitargeek changed the title [metacling] Implement TClingTypeInfo::Size with CppInterOp [metacling] Use GetSizeOfType from CppInterOp Mar 25, 2025
@guitargeek guitargeek requested a review from aaronj0 March 25, 2025 15:23
@github-actions
Copy link

Test Results

    19 files      19 suites   4d 10h 44m 9s ⏱️
 2 738 tests  2 736 ✅ 0 💤 2 ❌
50 059 runs  50 057 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit 7a253dd.

Copy link
Contributor

@aaronj0 aaronj0 left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for this PR!

@guitargeek guitargeek merged commit a510b5c into root-project:master Mar 26, 2025
20 of 23 checks passed
@guitargeek guitargeek deleted the TClingTypeInfo_Size branch March 26, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants