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

[bugzilla-daemon at cs.uiuc.edu]

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

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