[all-commits] [llvm/llvm-project] df579c: [libc++] Adds is_implemented function for new ftm ...

[Mark de Wever via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: df579ce4b6006ed23bb3a31f009269a1ba2587d4
      https://github.com/llvm/llvm-project/commit/df579ce4b6006ed23bb3a31f009269a1ba2587d4
  Author: Mark de Wever <koraq at xs4all.nl>
  Date:   2025-04-11 (Fri, 11 Apr 2025)

  Changed paths:
    M libcxx/test/libcxx/feature_test_macro/implemented_ftms.sh.py
    A libcxx/test/libcxx/feature_test_macro/is_implemented.sh.py
    M libcxx/utils/generate_feature_test_macro_components.py

  Log Message:
  -----------
  [libc++] Adds is_implemented function for new ftm generator. (#134538)

At the moment the ftm macro for __cpp_lib_to_chars will have the
following values:

standard_ftms: {
    "c++17": "201611L",
    "c++20": "201611L",
    "c++23": "201611L",
    "c++26": "201611L",
}

implemented_ftms: {
    "c++17": None,
}

This is an issue with the test whether the FTM is implemented it does:
  self.implemented_ftms[ftm][std] == self.standard_ftms[ftm][std]
This will fail in C++20 since implemented_ftms[ftm] does not have the
key c++20. This adds a new helper function and removes the None entries
when a FTM is not implemented.

---------

Co-authored-by: Louis Dionne <ldionne.2 at gmail.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications