Page MenuHomeFreeBSD
Differential D35889

Mk/Uses/compiler.mk: Support newer C++ standards
ClosedPublic
Actions

Authored by zirias on Jul 24 2022, 12:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 15, 2:53 PM
Unknown Object (File)
Tue, Sep 15, 7:14 AM
Unknown Object (File)
Mon, Sep 14, 11:16 PM
Unknown Object (File)
Mon, Sep 14, 10:11 PM
Unknown Object (File)
Mon, Sep 14, 5:58 PM
Unknown Object (File)
Mon, Sep 14, 12:54 PM
Unknown Object (File)
Mon, Sep 14, 7:28 AM
Unknown Object (File)
Mon, Sep 14, 7:23 AM
View All Files
Subscribers
mat

Details

Reviewers
jrm
bapt
tcberner
Group Reviewers
portmgr
Commits
R11:5ca22080ae08: Mk/Uses/compiler.mk: Support newer C++ standards
Summary
  • Add c++20-lang and c++2b-lang arguments
  • Respect LLVM_DEFAULT if it fullfills the requirements

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 46524
Build 43413: arc lint + arc unit

Event Timeline

zirias created this revision.Jul 24 2022, 12:45 PM
Herald added a reviewer: portmgr. · View Herald TranscriptJul 24 2022, 12:45 PM
Herald added a subscriber: mat. · View Herald Transcript
Harbormaster completed remote builds in B46519: Diff 108448.Jul 24 2022, 12:45 PM
zirias requested review of this revision.Jul 24 2022, 12:45 PM
zirias added a comment.Jul 24 2022, 12:47 PM

Remark: I'm not too sure about the support for newer C++ standards as this would probably also require support in STL/libstdc++...

But I really think compiler.mk should respect LLVM_DEFAULT when pulling in llvm from ports? Would of course require intensive testing!

zirias updated this revision to Diff 108450.Jul 24 2022, 1:01 PM

Fix min version for llvm

Checking for c++2b-lang must happen before c++20-lang to "win" regarding
the min version of llvm required.

Harbormaster completed remote builds in B46521: Diff 108450.Jul 24 2022, 1:01 PM
zirias updated this revision to Diff 108461.Jul 24 2022, 4:41 PM

Extend VALID_ARGS

forgot that ....

Harbormaster completed remote builds in B46524: Diff 108461.Jul 24 2022, 4:41 PM
bapt accepted this revision.Aug 24 2022, 6:58 AM

imho worth an exp-run

This revision is now accepted and ready to land.Aug 24 2022, 6:58 AM
tcberner accepted this revision.Sep 11 2022, 11:31 AM

ship it

Closed by commit R11:5ca22080ae08: Mk/Uses/compiler.mk: Support newer C++ standards (authored by zirias). · Explain WhySep 11 2022, 11:36 AM
This revision was automatically updated to reflect the committed changes.
zirias added a commit: R11:5ca22080ae08: Mk/Uses/compiler.mk: Support newer C++ standards.

Revision Contents
Changeset List

PathSize
Mk/
Uses/
42 lines

Diff 108461

View Options

Mk/Uses/compiler.mk

Loading...