[libc++] `std::ranges::clamp` violates [alg.clamp]p5 (original) (raw)

Skip to content

Appearance settings

- AI CODE CREATION
  * GitHub CopilotWrite better code with AI
  * GitHub SparkBuild and deploy intelligent apps
  * GitHub ModelsManage and compare prompts
  * MCP RegistryNewIntegrate external tools
- DEVELOPER WORKFLOWS
  * ActionsAutomate any workflow
  * CodespacesInstant dev environments
  * IssuesPlan and track work
  * Code ReviewManage code changes
- APPLICATION SECURITY
  * GitHub Advanced SecurityFind and fix vulnerabilities
  * Code securitySecure your code as you build
  * Secret protectionStop leaks before they start
- EXPLORE
  * Why GitHub
  * Documentation
  * Blog
  * Changelog
  * Marketplace

View all features

- BY COMPANY SIZE
  * Enterprises
  * Small and medium teams
  * Startups
  * Nonprofits
- BY USE CASE
  * App Modernization
  * DevSecOps
  * DevOps
  * CI/CD
  * View all use cases
- BY INDUSTRY
  * Healthcare
  * Financial services
  * Manufacturing
  * Government
  * View all industries

View all solutions

- EXPLORE BY TOPIC
  * AI
  * Software Development
  * DevOps
  * Security
  * View all topics
- EXPLORE BY TYPE
  * Customer stories
  * Events & webinars
  * Ebooks & reports
  * Business insights
  * GitHub Skills
- SUPPORT & SERVICES
  * Documentation
  * Customer support
  * Community forum
  * Trust center
  * Partners
- COMMUNITY
  * GitHub SponsorsFund open source developers
- PROGRAMS
  * Security Lab
  * Maintainer Community
  * Accelerator
  * Archive Program
- REPOSITORIES
  * Topics
  * Trending
  * Collections
- ENTERPRISE SOLUTIONS
  * Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  * GitHub Advanced SecurityEnterprise-grade security features
  * Copilot for BusinessEnterprise-grade AI features
  * Premium SupportEnterprise-grade 24/7 support
Pricing

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

llvm / llvm-project Public

Notifications You must be signed in to change notification settings
Fork15.5k
Star 35.9k
Code
Issues 5k+
Pull requests 5k+
Actions
Security
Insights

Additional navigation options

Closed

Closed

[libc++] std::ranges::clamp violates [alg.clamp]p5

#64717

Assignees

Labels

bugIndicates an unexpected problem or unintended behavior good first issuehttps://github.com/llvm/llvm-project/contribute libc++libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.rangesIssues related to ``

Description

opened

on Aug 15, 2023

[alg.clamp]p5 mandates the complexity requirements for std::ranges::clamp to be at most two comparisons and three applications of the projection. libc++ however applies the projection four times. See godbolt.

llvm-project/libcxx/include/__algorithm/ranges_clamp.h

Lines 40 to 43 in5ed62c7

if (std::invoke(__comp, std::invoke(__proj, __value), std::invoke(__proj, __low)))
return __low;
else if (std::invoke(__comp, std::invoke(__proj, __high), std::invoke(__proj, __value)))
return __high;

Metadata

Metadata

Assignees

ldionne

Labels

bugIndicates an unexpected problem or unintended behavior good first issuehttps://github.com/llvm/llvm-project/contribute libc++libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.rangesIssues related to ``

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions