Re: Breakage in Clang compile....

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9-2-2017 17:18, Bassam Tabbara wrote:
> Hi,
> 
> I’m curious are there specific advantages to compiling Ceph with
> clang vs. gcc? I understand that clang uses less memory, compiles
> faster, and has better errors/diagnostics, but as it relates to Ceph
> is there any specific advantages?
> 
> Is there a plan to switching to clang?

There are a few reasons for also using Clang:
 - Clang finds other types of issues in the code.
	I think the speed, mem-footprint advantages you mention are not
	(yet) the case.
 - FreeBSD has Clang as native compiler, and I'm preparing a Ceph port
   to FreeBSD.

Thanx for fixing this, I saw the PR fly by.

--WjW

> Thanks! Bassam
> 
>> On Feb 9, 2017, at 7:35 AM, Willem Jan Withagen <wjw@xxxxxxxxxxx>
>> wrote:
>> 
>> Hi,
>> 
>> I think this is a recent addition to the game? And Clang does not
>> really like it.
>> 
>> In file included from 
>> /home/jenkins/workspace/ceph-master/src/common/perf_counters.h:21: 
>> /home/jenkins/workspace/ceph-master/src/common/perf_histogram.h:80:9:
>>
>> 
error: array initializer must be an initializer list
>> : m_axes_config(other.m_axes_config) { ^ 
>> /home/jenkins/workspace/ceph-master/src/common/perf_counters.h:98:29:
>>
>> 
note: in instantiation of member function
>> 'PerfHistogram<2>::PerfHistogram' requested here 
>> histogram.reset(new PerfHistogram<>(*other.histogram)); ^ 1 error
>> generated.
>> 
>> --WjW -- To unsubscribe from this list: send the line "unsubscribe
>> ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx 
>> More majordomo info at
>> https://urldefense.proofpoint.com/v2/url?u=http-3A__vger.kernel.org_majordomo-2Dinfo.html&d=DwICaQ&c=8S5idjlO_n28Ko3lg6lskTMwneSC-WqZ5EBTEEvDlkg&r=BTMd2ANcDl5P_nTkEam5zzywWdGjHoaoXy4JMG_yHPA&m=PzCl2U7BH52mycDw3PiC7LaCbGCCTewMNBUqu8gGKIQ&s=gR4xXKHg4AFcwEccrIs-KoM7bjGw0aL9CnVF_PQE_fo&e=
>> 
> 

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux