[LLVMbugs] [Bug 12320] New: some 'configure' problems

0 views

Skip to first unread message

bugzill...@llvm.org

unread,

Mar 21, 2012, 3:13:07 AM3/21/12

to llvm...@cs.uiuc.edu

http://llvm.org/bugs/show_bug.cgi?id=12320

Bug #: 12320
Summary: some 'configure' problems
Product: new-bugs
Version: unspecified
Platform: All
OS/Version: All
Status: NEW
Severity: enhancement
Priority: P
Component: new bugs
AssignedTo: unassig...@nondot.org
ReportedBy: s...@tbricks.com
CC: llvm...@cs.uiuc.edu
Classification: Unclassified

Created attachment 8246
--> http://llvm.org/bugs/attachment.cgi?id=8246
suggested patch

While bootstrapping clang w/ libc++ on Solaris I found several problems
in "configure" scripts.

1) Typo in std::isinf() detection in autoconf/m4/func_isinf.m4
Duplicated in projects/sample/autoconf/m4/func_isinf.m4

2) When configuring clang to build using libc++ I think '-stdlib=libc++'
should be used for all C++ configure tests

3) 'projects/sample' has its own configure script which lacks libc++ support

I attached patch, please review/commit.

--
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
LLVMbugs mailing list
LLVM...@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

bugzill...@llvm.org

unread,

Mar 25, 2012, 10:10:02 PM3/25/12

to llvm...@cs.uiuc.edu

http://llvm.org/bugs/show_bug.cgi?id=12320

Eric Christopher <echr...@gmail.com> changed:

--- Comment #2 from Eric Christopher <echr...@gmail.com> 2012-03-25 21:10:02 CDT ---
Dmitri:

I applied most of it here:

M configure
M projects/sample/Makefile.llvm.config.in
M projects/sample/Makefile.llvm.rules
M projects/sample/configure
M projects/sample/autoconf/m4/func_isinf.m4
M projects/sample/autoconf/m4/huge_val.m4
M projects/sample/autoconf/configure.ac
M autoconf/m4/func_isinf.m4
M autoconf/m4/huge_val.m4
r153425 = 88376b6cfead1c3de0c761ce33ea03544d837b94 (refs/remotes/origin/master)

the changes to CXXFLAGS don't seem to be correct, or at least interfere with
finding isInf so I avoided them at the moment. Feel free to submit another
patch for that if you can get it to build.