[LLVMbugs] [Bug 5664] New: DISABLE_INLINE build fix

bugzilla-daemon at cs.uiuc.edu bugzilla-daemon at cs.uiuc.edu
Tue Dec 1 19:42:17 PST 2009


           Summary: DISABLE_INLINE build fix
           Product: libraries
           Version: trunk
          Platform: PC
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Backend: ARM
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: alp at nuanti.com
                CC: llvmbugs at cs.uiuc.edu

DISABLE_INLINE needs to be before the declaration when building with MSVC.

Error   1       error C2144: syntax error : 'int' should be preceded by ';'    
c:\Documents and Settings\alp\My Documents\Visual Studio
2008\Projects\llvm\lib\Target\ARM\ARMBaseInstrInfo.cpp 423

Fix (only tested on MSVC):

--- a/lib/Target/ARM/ARMBaseInstrInfo.cpp
+++ b/lib/Target/ARM/ARMBaseInstrInfo.cpp
@@ -419,8 +419,9 @@ bool ARMBaseInstrInfo::isPredicable(MachineInstr *MI) const

 /// FIXME: Works around a gcc miscompilation with -fstrict-aliasing
 static unsigned getNumJTEntries(const std::vector<MachineJumpTableEntry> &JT,
-                                unsigned JTI) DISABLE_INLINE;
+                                unsigned JTI);
 static unsigned getNumJTEntries(const std::vector<MachineJumpTableEntry> &JT,
                                 unsigned JTI) {
   return JT[JTI].MBBs.size();

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.

More information about the llvm-bugs mailing list