Skip to content

[SSAF] Fix Expected return type in TypeConstrainedPointers deserialization (gcc 7.5.0) - #22727

Merged
KornevNikita merged 1 commit into
intel:syclfrom
KornevNikita:fix-gcc-750
Jul 23, 2026
Merged

[SSAF] Fix Expected return type in TypeConstrainedPointers deserialization (gcc 7.5.0)#22727
KornevNikita merged 1 commit into
intel:syclfrom
KornevNikita:fix-gcc-750

Conversation

@KornevNikita

Copy link
Copy Markdown
Contributor

GCC 7.5.0 fails to compile this code. Use explicit upcasts from std::unique_ptr to std::unique_ptr in deserializeSummary and deserializeAnalysisResult. This resolves a compilation error where llvm::Expectedstd::unique_ptr could not be constructed from unique_ptr of derived summary/result types.


See: llvm/llvm-project#211331
This file has been renamed in the upstream. The PR above should fix the issue in the upstream.
This one is to fix the toolchain build with gcc 7.5.0 before pulldown.

…ation (gcc 7.5.0)

GCC 7.5.0 fails to compile this code. Use explicit upcasts from std::unique_ptr to std::unique_ptr in deserializeSummary and deserializeAnalysisResult. This resolves a compilation error where llvm::Expected<std::unique_ptr> could not be constructed from unique_ptr of derived summary/result types.

---
See: llvm/llvm-project#211331
This file has been renamed in the upstream. The PR above should fix the
issue in the new file. This one is to fix the toolchain build with gcc 7.5.0 before pulldown.
@KornevNikita
KornevNikita requested a review from a team as a code owner July 22, 2026 17:46
@KornevNikita

Copy link
Copy Markdown
Contributor Author

@intel/dpcpp-cfe-reviewers I’d really appreciate a quick review; the patch is trivial.

@elizabethandrews elizabethandrews left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks ok to me but @premanandrao can you also take a look so I am not missing any unexpected consequences

@sys-ce-bb

Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@premanandrao premanandrao left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks okay to me.

@KornevNikita
KornevNikita merged commit 857a772 into intel:sycl Jul 23, 2026
31 checks passed
@KornevNikita
KornevNikita deleted the fix-gcc-750 branch July 23, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants