[Bug 133] New: setting CFLAGS usually breaks inlined

http://sablevm.org/bugs/show_bug.cgi?id=133

           Summary: setting CFLAGS usually breaks inlined
           Product: SableVM
           Version: trunk
          Platform: PC
               URL: http://sablevm.org/bugs/attachment.cgi?id=37&action=view
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: default
        AssignedTo: [email protected]
        ReportedBy: [email protected]
         QAContact: [email protected]


Our compilation of inlined relies on fact that CFLAGS already contain -O2,
which is the default set by configure script if CFLAGS was not set.

However usually users setting CFLAGS to include additional include directories
don't include the necessary -O2 option, so the compiler does not inline
functions, which breaks the inline-threaded engine.

I think we should prepend -O2 to the CFLAGS no matter what.  This way the
user's -O* parameter would occur later in CFLAGS and take precedence.

Accidentally the (partial) patch for BUG 111 seem to also contains the
solution for this bug.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are the QA contact for the bug, or are watching the QA contact.