Closed JakeHillion closed 1 month ago
This pull request was exported from Phabricator. Differential Revision: D61268962
This pull request was exported from Phabricator. Differential Revision: D61268962
This pull request was exported from Phabricator. Differential Revision: D61268962
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 60.78%. Comparing base (
ed5fcfe
) to head (e55fd33
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary: The internal build is moving to LLVM-17 from LLVM-15. For our code there are two required changes to get it building: a header has been renamed and the signature of a function has changed.
Checked which LLVM major version these changes apply in and both are 16, macro gated accordingly so we maintain compatibility with both LLVM-15 and LLVM-17. Both versions build internally.
Differential Revision: D61268962