Skip to content

Fix clang build with GCC 7.5#22415

Open
fda0 wants to merge 1 commit into
intel:syclfrom
fda0:fix_gcc_7.5
Open

Fix clang build with GCC 7.5#22415
fda0 wants to merge 1 commit into
intel:syclfrom
fda0:fix_gcc_7.5

Conversation

@fda0

@fda0 fda0 commented Jun 24, 2026

Copy link
Copy Markdown

GCC 7.5 rejects two newer C++ idioms:

  • LifetimeSafety/FactsGenerator: replace ArrayRef CTAD with an explicit
    ArrayRef<const Expr *> to avoid GCC 7's broken brace-init deduction.
  • ScalableStaticAnalysisFramework: add explicit std::move() on
    unique_ptr returns (pre-P1825, GCC 7 won't implicitly move).

No behavior change; restores compatibility with GCC 7.5.

GCC 7.5 rejects two newer C++ idioms:

- LifetimeSafety/FactsGenerator: replace ArrayRef CTAD with an explicit
  ArrayRef<const Expr *> to avoid GCC 7's broken brace-init deduction.
- ScalableStaticAnalysisFramework: add explicit std::move() on
  unique_ptr<Derived> returns (pre-P1825, GCC 7 won't implicitly move).

No behavior change; restores compatibility with GCC 7.5.
@fda0 fda0 requested a review from a team as a code owner June 24, 2026 13:23

@bader bader 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 change is not related to SYCL. According to the guidelines, this patch must go to https://github.com/llvm/llvm-project/ repository.

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.

2 participants