rippled icon indicating copy to clipboard operation
rippled copied to clipboard

Switch from Clang 14 to Clang 16

Open bthomee opened this issue 9 months ago • 1 comments

High Level Overview of Change

This change switches from Clang 14 to Clang 16.

Context of Change

Clang 14 has not been in development since 2022, and it is lagging in C++ 20 that we now commonly use. Antithesis also requires Clang 16 and the corresponding CI pipeline requires building from scratch for that compiler. With this switch we optimize our CI time and ensure we are ready for more modern C++ versions.

Type of Change

  • [X] Refactor (non-breaking change that only restructures code)

bthomee avatar Mar 27 '25 15:03 bthomee

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.0%. Comparing base (e429455) to head (5ca72a9).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##           develop   #5369     +/-   ##
=========================================
- Coverage     78.1%   78.0%   -0.1%     
=========================================
  Files          795     795             
  Lines        68567   68663     +96     
  Branches      8286    8375     +89     
=========================================
- Hits         53545   53540      -5     
- Misses       15022   15123    +101     

see 98 files with indirect coverage changes

Impacted file tree graph

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Apr 08 '25 07:04 codecov[bot]

We will soon start using a series of Ubuntu, Red Hat, and Debian images with varying GCC and Clang compilers enabled within, and as such this PR isn't necessary anymore in its current form.

bthomee avatar Jun 24 '25 17:06 bthomee