LLVM under MS VC++ 2005

Project Infrastructure LLVM Dev List Archives
1

I missed two files from the LLVM source code proper. These need reviewing and commiting. They are trivial changes that should not influence any other builds. Basically these functions are missing a return statement which Visual Studio 2005 Beta flags as an error rather than as a warning. I have put in dummy constructors in return statements to allow CodeGen and Target libraries to build.

The files :-

\llvm\lib\codegen\selectiondag\selectiondagisel.cpp

llvm::TargetLowering::LowerVAStart
llvm::TargetLowering::LowerVAArgNext
llvm::TargetLowering::LowerOperation

\llvm\lib\target\targetframeinfo.cpp

llvm::TargetFrameInfo::adjustAlignment
llvm::TargetFrameInfo::getIncomingArgOffset
llvm::TargetFrameInfo::getOutgoingArgOffset
llvm::TargetFrameInfo::getFirstAutomaticVarOffset
llvm::TargetFrameInfo::getRegSpillAreaOffset
llvm::TargetFrameInfo::getTmpAreaOffset
llvm::TargetFrameInfo::getDynamicAreaOffset

Note I have not commited anything to CVS.

Aaron

SelectionDAGISel.cpp (39.8 KB)

TargetFrameInfo.cpp (2.21 KB)

2

Thanks for the patches!

Commits here:

http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20050214/024164.html
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20050214/024165.html

3

I missed two files from the LLVM source code proper. These need
reviewing and commiting. They are trivial changes that should not
influence any other builds. Basically these functions are missing a
return statement which Visual Studio 2005 Beta flags as an error
rather than as a warning. I have put in dummy constructors in return
statements to allow CodeGen and Target libraries to build.

The files :-

    \llvm\lib\codegen\selectiondag\selectiondagisel.cpp
    \llvm\lib\target\targetframeinfo.cpp

Thanks for the patches!

Commits here:

http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20050214/024164.html
http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20050214/024165.html

Thank you.

Hopefully someone will be able to test the main llvm-VS2005 patch soon.

Cheers,

Aaron