[LLVMbugs] [Bug 8172] New: configure --help in release_28 branch erroneously says assertions are enabled by default

[bugzilla-daemon at llvm.org]

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

           Summary: configure --help in release_28 branch erroneously says
                    assertions are enabled by default
           Product: new-bugs
           Version: 2.8
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: new bugs
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: dimitry at andric.com
                CC: llvmbugs at cs.uiuc.edu


Using r114020 in the release_28 branch, configure --help says that
assertions are enabled by default:

$ ./configure --help
`configure' configures llvm 2.8rc to adapt to many kinds of systems.
...
  --enable-assertions     Compile with assertion checks enabled (default is
                          YES)

However, the script contains:

# Check whether --enable-assertions was given.
if test "${enable_assertions+set}" = set; then
  enableval=$enable_assertions;
else
  enableval="no"
fi

so actually, assertions are disabled by default.  Even
autoconf/configure.ac says so:

dnl --enable-assertions : check whether they want to turn on assertions or not:
AC_ARG_ENABLE(assertions,AS_HELP_STRING(
  --enable-assertions,[Compile with assertion checks enabled (default is
NO)]),, enableval="no")
if test ${enableval} = "yes" ; then
  AC_SUBST(DISABLE_ASSERTIONS,[[]])
else
  AC_SUBST(DISABLE_ASSERTIONS,[[DISABLE_ASSERTIONS=1]])
fi

Maybe configure needs to be regenerated?

-- 
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.