Opened 4 years ago

Closed 2 years ago

#62637 closed defect (fixed)

creduce @2.10.0 : depends on clang-8.0 which doesn't work on Big Sur

Reported by: c-kloukinas Owned by: larryv (Lawrence Velázquez)
Priority: Normal Milestone:
Component: ports Version: 2.6.4
Keywords: BigSur Cc: mascguy (Christopher Nielsen)
Port: creduce

Description

Can it be built with a newer version of clang that's supported in BigSur?

Change History (6)

comment:1 Changed 4 years ago by jmroot (Joshua Root)

Cc: larryv@… removed
Owner: set to larryv
Status: newassigned
Summary: creduce @2.10.0 : depends on broken and abandoned (BigSur) clang-8.0creduce @2.10.0 : depends on clang-8.0 which doesn't work on Big Sur

comment:2 Changed 4 years ago by kencu (Ken)

we were supposed to update the clang dep to 9.0 when we upgraded to creduce 2.10

<https://github.com/csmith-project/creduce/blob/master/INSTALL.md>

and that is working on BigSur Intel <http://packages.macports.org/clang-9.0/>.

For BigSur arm64, we should use llvm-11/clang-11. There is a creduce branch for that, and a fairly simple patch from that branch here <https://github.com/csmith-project/creduce/commit/3e1f5a1ec1365fafae965c97603221c9ccbc2d61> that does the actual llvm-11 fix.

So looks all doable.

comment:3 Changed 2 years ago by mascguy (Christopher Nielsen)

Cc: mascguy added

comment:4 Changed 2 years ago by mascguy (Christopher Nielsen)

comment:5 Changed 2 years ago by mascguy (Christopher Nielsen)

It looks like separate patches are needed, for LLVM versions beyond 11.

LLVM 12:

https://github.com/csmith-project/creduce/commit/05aa5659c055860e84ec9f53b19e1228b34510a1.patch

LLVM 13 requires the aforementioned patch for 12, along with:

https://github.com/csmith-project/creduce/commit/4ae77d1e101135a6aea310b30c19e2af968b7bf6.patch

comment:6 Changed 2 years ago by Christopher Nielsen <mascguy@…>

Resolution: fixed
Status: assignedclosed

In 73e1c3434058a4cbd61502883be7144aedbb21cf/macports-ports (master):

creduce: add variants llvm90, llvm11

  • Auto-select llvm11 for Big Sur and later
  • Also migrate default from llvm 8 to 9

Fixes: #62637

Note: See TracTickets for help on using tickets.