This is an archive of the discontinued LLVM Phabricator instance.

Add Cmake Check for atomic<double> intrinsic
Needs ReviewPublic

Authored by raj.khem on Nov 23 2019, 12:00 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

On some architectures e.g. x86/32bit gcc decides to inline calls to
double atomic variables but clang does not and defers it to libatomic
therefore detect if clang can use built-ins for atomic<double> if not
then link libatomic, this helps building clangd for x86 on linux systems
with gcc runtime

Diff Detail

Event Timeline

raj.khem created this revision.Nov 23 2019, 12:00 PM

Please upload all patches with full context (-U99999)

raj.khem updated this revision to Diff 230783.Nov 23 2019, 7:23 PM

Can someone review this please?