[all-commits] [llvm/llvm-project] b81c69: [libc++][NFC] Add a few explicit 'inline' keywords...

Louis Dionne via All-commits all-commits at lists.llvm.org
Wed Dec 13 07:17:40 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b81c69415e47bcf59801cb751eed08a6d1cb23f1
      https://github.com/llvm/llvm-project/commit/b81c69415e47bcf59801cb751eed08a6d1cb23f1
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2023-12-13 (Wed, 13 Dec 2023)

  Changed paths:
    M libcxx/include/__chrono/day.h
    M libcxx/include/__chrono/hh_mm_ss.h
    M libcxx/include/__chrono/month.h
    M libcxx/include/__chrono/monthday.h
    M libcxx/include/__chrono/weekday.h
    M libcxx/include/__chrono/year_month.h
    M libcxx/include/__chrono/year_month_day.h
    M libcxx/include/__variant/monostate.h
    M libcxx/include/cmath
    M libcxx/include/complex
    M libcxx/include/cstddef

  Log Message:
  -----------
  [libc++][NFC] Add a few explicit 'inline' keywords, mostly in <chrono> (#75234)

Even though constexpr implicitly makes functions inline, we try not to
rely on this implicit effect in the code base. We are mostly consistent
about using `inline` on non-template free-functions to make it clear
that we don't have an ODR violation.

This patch simply fixes a few places where we didn't explicitly use
inline on non-template free functions, presumably because they were
constexpr.

Fixes #75227




More information about the All-commits mailing list