Ticket #10685 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Get clang working on linux for Kernel

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Ross Whitfield

Description

As it is the start of all things...get Kernel to build with gnu-clang.

Change History

comment:1 Changed 6 years ago by Peter Peterson

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Pete Peterson

  • Status changed from assigned to inprogress

Refs #10685. Got kernel building with clang on f20.

Changeset: 4c349f9b916e4d0bf2e7427e63b87765be6a2149

comment:3 Changed 6 years ago by Pete Peterson

Refs #10685. Squasing compiler warnings.

Changeset: 2d55a004698bed91e38429f25e17268c9d50a773

comment:4 Changed 6 years ago by Pete Peterson

Re #10685. Changed return type to match what is actually returned.

Changeset: f25befd52136797a1e99a016e47505d1eef647ea

comment:5 Changed 6 years ago by Pete Peterson

Re #10685. Bringing back some old code.

Changeset: ce95d2679675a68397cdf04ff9a0f0d27fbf5c80

comment:6 Changed 6 years ago by Pete Peterson

  • Status changed from inprogress to verify
  • Resolution set to fixed

This is being verified as ​pull request #86.

comment:7 Changed 6 years ago by Ross Whitfield

  • Status changed from verify to verifying
  • Tester set to Ross Whitfield

comment:8 Changed 6 years ago by Martyn Gigg

I was having a quick play with the include-what-you-use tool after merging these changes locally. It seems that clang requires about ~12Gb of memory to compile Kernel! I've opened #10700(​http://trac.mantidproject.org/mantid/ticket/10700) to investigate as it's not these changes that cause it.

comment:9 Changed 6 years ago by Pete Peterson

retest this please

comment:10 Changed 6 years ago by Ross Whitfield

This builds and runs using clang-3.4 but fails to build for me using clang-3.5. Something that will need to be looked at in the future.

comment:11 Changed 6 years ago by Ross Whitfield

  • Status changed from verifying to closed

Merge pull request #86 from mantidproject/feature/10685_gnu_clang

Get clang working on linux for Kernel

Full changeset: ​3507e3aa6c2fff819478aa6c5623f9ba6fb9e487

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​11527

Note: See TracTickets for help on using tickets.