[cfe-commits] r146051 - /cfe/trunk/runtime/compiler-rt/Makefile

[Evgeniy Stepanov]

On Thu, Dec 8, 2011 at 1:07 AM, Daniel Dunbar <daniel at zuster.org> wrote:

> On Wed, Dec 7, 2011 at 1:01 PM, Kostya Serebryany <kcc at google.com> wrote:
> > Thanks!
> >
> > Question:
> > There seem to be no way to build 32- and 64-bit linux libraries with a
> > single make command.
> > Am I right?
>
> Yes.
>
> > Do you plan to change this?
>
> I would like to, but it's a tad tricky with the current level of build
> support we have. This is also a cumbersome area to test because it
> really varies quite a lot based on the host Linux system, and there is
> so much diversity amongst distributions. I really wish we had a good
> farm of different common Linux distros available.
>
> > This change requires to fix the asan driver
> > in tools/clang/lib/Driver/Tools.cpp, I'll prepare a patch unless you
> have it
> > ready.
>
> I'll fix it, I'm in the area.
>
> > Evgeniy, could you please prepare a similar patch for linux/arm (take
> > r146051 and r145950 as an example)
>
> This is going to be hard. Unless I am mistaken, there is little to no
> standard for how to find the ARM toolchain to use to cross compile. Am
> I mistaken?
>
> My long term plan for fixing this is to make it blocked on good cross
> compilation support in Clang, but I understand that is not ideal if
> you want ASAN/ARM available in Clang ASAP.
>

We mostly want it on Android/ARM. This might be easier, as there is much
less variety in the toolchain(s).

I can make it work (kind of) with a long list of -B, -I and -L. For both
Android/ARM and Linux/ARM. A slightly broken version of that can be found
in Makefile.old. Some of the paths can be replaced with --sysroot, but I've
got no idea how to get rid of, for
example, -I$(CROSSTOOL)/arm-unknown-linux-gnueabi/include/c++/4.4.4


>
>  - Daniel
>
> >
> > --kcc
> >
> >
> >
> > On Wed, Dec 7, 2011 at 11:35 AM, Daniel Dunbar <daniel at zuster.org>
> wrote:
> >>
> >> Author: ddunbar
> >> Date: Wed Dec  7 13:35:10 2011
> >> New Revision: 146051
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=146051&view=rev
> >> Log:
> >> runtime/Linux: Include the profile and ASAN libs on x86.
> >>
> >> Modified:
> >>    cfe/trunk/runtime/compiler-rt/Makefile
> >>
> >> Modified: cfe/trunk/runtime/compiler-rt/Makefile
> >> URL:
> >>
> http://llvm.org/viewvc/llvm-project/cfe/trunk/runtime/compiler-rt/Makefile?rev=146051&r1=146050&r2=146051&view=diff
> >>
> >>
> ==============================================================================
> >> --- cfe/trunk/runtime/compiler-rt/Makefile (original)
> >> +++ cfe/trunk/runtime/compiler-rt/Makefile Wed Dec  7 13:35:10 2011
> >> @@ -93,10 +93,12 @@
> >>
> >>  # We currently only try to generate runtime libraries on x86.
> >>  ifeq ($(ARCH),x86)
> >> -RuntimeLibrary.linux.Configs += full-i386
> >> +RuntimeLibrary.linux.Configs += \
> >> +       full-i386 profile-i386 asan-i386
> >>  endif
> >>  ifeq ($(ARCH),x86_64)
> >> -RuntimeLibrary.linux.Configs += full-x86_64
> >> +RuntimeLibrary.linux.Configs += \
> >> +       full-x86_64 profile-x86_64 asan-x86_64
> >>  endif
> >>
> >>  endif
> >>
> >>
> >> _______________________________________________
> >> cfe-commits mailing list
> >> cfe-commits at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> >
> >
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111208/5ef8fd06/attachment.html>