Differential D137498

[libc++][math.h][NFC] Reformat functions
ClosedPublic
Actions

Authored by philnik on Nov 5 2022, 1:48 PM.

Tags

Subscribers

Details

Reviewers

ldionne
Mordante
var-const
huixie90
EricWF

Group Reviewers

Restricted Project

Commits

rGe1e1cad6fd75: [libc++][math.h][NFC] Reformat functions

Summary

This is just to make to follow-up diffs simpler

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

philnik created this revision.Nov 5 2022, 1:48 PM

Herald added a project: Restricted Project. · View Herald TranscriptNov 5 2022, 1:48 PM

philnik requested review of this revision.Nov 5 2022, 1:48 PM

Herald added a project: Restricted Project. · View Herald TranscriptNov 5 2022, 1:48 PM

Herald added a reviewer: Restricted Project. · View Herald Transcript

Herald added a subscriber: libcxx-commits. · View Herald Transcript

philnik added a parent revision: D136908: [libc++] Remove duplication in math_h.pass.cpp and improve coverage.Nov 5 2022, 1:49 PM

philnik added a child revision: D135781: [libc++] Assume that builtins for math.h functions are available.

Harbormaster completed remote builds in B196319: Diff 473460.Nov 5 2022, 3:07 PM

This is from clang-format? LGTM then.

This revision is now accepted and ready to land.Nov 6 2022, 2:38 PM

LGTM

Closed by commit rGe1e1cad6fd75: [libc++][math.h][NFC] Reformat functions (authored by philnik). · Explain WhyNov 21 2022, 1:52 PM

This revision was automatically updated to reflect the committed changes.

philnik added a commit: rGe1e1cad6fd75: [libc++][math.h][NFC] Reformat functions.

philnik removed a child revision: D135781: [libc++] Assume that builtins for math.h functions are available.Nov 21 2022, 1:53 PM

Revision Contents
Changeset List

Path

Size

libcxx/

include/

468 lines

Diff 476993

libcxx/include/math.h

Loading...