clang package is missing includes

Bug #903509 reported by meh
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
clang (Debian)
Fix Released
Unknown
clang (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

/usr/include/clang/3.0/include/* should be part of 'clang' and not 'libclang-dev', while the other headers are fine in that package (for libclang.so).

These headers contains the SIMD intrinsics like xmmintrin.h
Without libclang-dev clang now includes gcc's version, which in return uses __builtin_*()s.

And that looks like this:

/usr/lib/gcc/x86_64-linux-gnu/4.6/include/xmmintrin.h:102:19: error: use of
      undeclared identifier '__builtin_ia32_addss'
  return (__m128) __builtin_ia32_addss ((__v4sf)__A, (__v4sf)__B);

Installing libclang-dev fixes this as clang picks those up automatically.

Related branches

Revision history for this message
Sylvestre Ledru (sylvestre) wrote :

Which version of clang are you using ?

Revision history for this message
meh (meh-deactivatedaccount) wrote :

3.0 from the official precise repo

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in clang (Ubuntu):
status: New → Confirmed
Revision history for this message
Sylvestre Ledru (sylvestre) wrote :

this has been fixed by version 3.0-4

Changed in clang (Debian):
status: Unknown → Fix Released
Revision history for this message
Anders Kaseorg (andersk) wrote :

Here’s an debdiff to merge clang 3.0-5 from unstable. I also fixed the DetectLinuxDistro logic so that it won’t automatically break when precise+1 comes out.

(Not tested yet; it’s currently waiting to build in https://launchpad.net/~anders-kaseorg/+archive/ppa .)

Revision history for this message
Anders Kaseorg (andersk) wrote :

Works for me, subscribing sponsors.

Changed in clang (Ubuntu):
assignee: nobody → Anders Kaseorg (anders-kaseorg)
assignee: Anders Kaseorg (anders-kaseorg) → Fabrice Coutadeur (fabricesp)
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

@Anders: Could you please rebase your merge on 3.0-6 (I'm not able to download 3.0-5 anymore) and post the debdiff between Debian and Ubuntu? Generally speaking, it's easier to review for sponsors when you have the diff between Debian and Ubuntu rather than a full Ubuntu -> Ubuntu diff.

Also, I've seen that the control file is generated from files in debian/control.in directory, so update-maintainer fails: you should update debian/control.in/source file with the Ubuntu maintainer.

Thanks for your work!

Fabrice

Changed in clang (Ubuntu):
assignee: Fabrice Coutadeur (fabricesp) → nobody
assignee: nobody → Anders Kaseorg (anders-kaseorg)
Revision history for this message
Anders Kaseorg (andersk) wrote :

Sure. But FYI,

> (I'm not able to download 3.0-5 anymore)
$ dget https://launchpad.net/debian/+archive/primary/+files/clang_3.0-5.dsc

> update-maintainer fails
Not a regression, but fixed now.

Here’s a diff from 3.0-6.

Changed in clang (Ubuntu):
assignee: Anders Kaseorg (anders-kaseorg) → Fabrice Coutadeur (fabricesp)
status: Confirmed → In Progress
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

Uploaded: thank you very much for your work!

Changed in clang (Ubuntu):
assignee: Fabrice Coutadeur (fabricesp) → nobody
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package clang - 3.0-6ubuntu1

---------------
clang (3.0-6ubuntu1) precise; urgency=low

  * Merge with Debian (LP: #903509); remaining changes:
    - Set Ubuntu as vendor name instead of Debian.
    - Make the `precise' code name known.
    - Update linker flags for `precise'.
    - Set dynamic linker name for armhf, recognize arm-linux-gnueabihf triplet.
    - Remove auto-generated debian_path.h.
  * debian/patches/23-ubuntu-name.diff: Assume unknown Ubuntu releases
    behave like precise, until we know otherwise.
 -- Anders Kaseorg <email address hidden> Fri, 02 Mar 2012 02:14:00 -0500

Changed in clang (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Fabrice Coutadeur (fabricesp) wrote :

@Anders: By the way, could you please forward the Ubuntu changes to Debian? The one about precise and the one about arm, at least.

Thanks again!

Fabrice

Revision history for this message
Brad Parker (cbparker) wrote :

I'm still having the same problem in trusty, I cannot compile any programs with clang as it complains about gcc's vector includes which it shouldn't be using. libclang-dev does not install any files for me either, so I'm not sure what the fix is.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.