This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: The actual LLVM integration patch


On Tue, 29 Nov 2005, Andrew Pinski wrote:
I threw the current version of the patch up here:
http://nondot.org/sabre/llvm-gcc-4.0-patch.tar.gz

A couple of comments.


getIntegerType is really badly.  It seems better to use
the mode to detect the type.

Also maping 128bit fp type to {double, double} seems wrong
for almost all targets except for PPC and MIPS which uses almost
the same 128bit fp encoding.

Thanks for the feedback! I will address these in the patch I actually propose for submission.


-Chris

--
http://nondot.org/sabre/
http://llvm.org/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]