similar to: [LLVMdev] diffs for vc7.1

On Sep 15, 2004, at 8:40 AM, Henrik Bach wrote: > Hi, > > Has some one committed these patches into CVS? Not yet... checked 10 minutes ago... I'm attaching the cvs diff -u version of them (I've read Reid Spencer post about it) > Paolo, thanks for the patches. In the near future I'll try to compile > them. > > I've installed STLport 6.4.2 with vc6. May I

On Wed, 2004-09-15 at 00:17, Paolo Invernizzi wrote: > On Sep 15, 2004, at 8:40 AM, Henrik Bach wrote: > > > Hi, > > > > Has some one committed these patches into CVS? > > Not yet... checked 10 minutes ago... Hopefully Chris will get to it today (Wednesday). > I'm attaching the cvs diff -u version of them (I've read Reid Spencer > post about it)

On Fri, 3 Sep 2004, Paolo Invernizzi wrote: > I can confirm that both are compiled properly: Ok. > for (BasicBlock::iterator I = H->begin; isa<PHINode>(I); I++) { > PHINode *PN = cast<PHINode(I); > .... > } > > I'll make a patch for whatever solution do you prefer (this problem is > a showstopper for more than a dozen files...) I prefer this

>From: Paolo Invernizzi <arathorn at fastwebnet.it> >Date: Thu, 16 Sep 2004 11:18:00 +0200 >If VC6 is not doing something wrong with templates... ;-/ As far as I know is template handling a part of some C++ ISO standard. Then, MS should adhere to this... However, what you objects against is that MS didn't implement STL the way STL should be implemented. And the reason to

On Fri, Sep 03, 2004 at 03:01:01PM -0500, Anshu Dasgupta wrote: > ><snip> > >for (BasicBlock::iterator I = Dest->begin(); PHINode *PN = > >dyn_cast<PHINode>(I); ++I) > > visitPHINode(*PN); > ><snip> > > > >build_vc71\lib\Transforms\Scalar\SCCP.cpp(202) : error C2275: > >'llvm::PHINode' : illegal use of this type as

I can confirm that both are compiled properly: A) PHINode *PN; for (BasicBlock::iterator I = H->begin(); PN = dyn_cast<PHINode>(I); I++) .... B) for (BasicBlock::iterator I = H->begin; isa<PHINode>(I); I++) { PHINode *PN = cast<PHINode(I); .... } I'll make a patch for whatever solution do you prefer (this problem is a showstopper for more than a dozen

>From: Paolo Invernizzi <arathorn at fastwebnet.it> >Date: Thu, 16 Sep 2004 10:20:39 +0200 > >I'm using scons to generate that files from .in files. I implemented in it >the configure check regarding iterators, hash and so on... >something like: > Hey, you've found the tool that makes it possible to generically reading Makefiles... Cool - The tool I've

I'll wait for the research. We should try, as much as possible, to make it work with just what the compiler provides and without third party packages. Thanks, reid. On Fri, 2004-09-24 at 07:46, Jeff Cohen wrote: > I added the include of cstudio and it fails with plain VC7.1; the file > does not exist. > > Add it for now. If it is impossible to build with VC7.1 and without

Hi all, Here the first bunch of patch for compiling part of LLVM under win32 with MSVC 7.1: * Trivial addings (I hope!): - #include <string> at top of: llvm\include\llvm\ExecutionEngine\ExecutionEngine.h(78) : error C2039: 'string' : is not a member of '_STL' - #include <algorithm> at top of: llvm\lib\CodeGen\LiveIntervalAnalysis.cpp(639) : error C2039:

Someone needs to adjudicate on whether I add the #include of <cstdio> or not. I can't test this so, Paolo/Henrik/Jeff, please let me know if I need to add it. Thanks, Reid. On Fri, 2004-09-24 at 07:08, Jeff Cohen wrote: > But I compiled that under vc7.1 as it was! > > On Fri, 24 Sep 2004 15:19:22 +0200 > Paolo Invernizzi <arathorn at fastwebnet.it> wrote: > >

I checked right now that it compiles also with #include <iostream> Jeff, can you test it with plain VC? --- Paolo Invernizzi On Sep 24, 2004, at 4:52 PM, Reid Spencer wrote: > I'll wait for the research. We should try, as much as possible, to make > it work with just what the compiler provides and without third party > packages. > > Thanks, > > reid. > >

On Sep 15, 2004, at 8:04 PM, Jeff Cohen wrote: > I cannot find any specific instructions for uninstalling the VC++ 2005 > Express Edition. There is this README file: > http://lab.msdn.microsoft.com/express/visualc/readme/default.aspx. As > it isn't the full Visual Studio it probably isn't messing around with > Windows system files. > > I do know for a fact that the

On Sep 15, 2004, at 7:29 PM, Jeff Cohen wrote: > Just make sure you read the uninstall instructions before installing. > Microsoft has been known to essentially require the reformatting your > boot partition in order to uninstall beta versions of VS. And, of > course, it is required that you uninstall the beta before installing a > newer beta or the release version. Urrggg! I

I added the include of cstudio and it fails with plain VC7.1; the file does not exist. Add it for now. If it is impossible to build with VC7.1 and without STLPort, then there really isn't a choice. I have to research the issue myself to see if it's possible to avoid using STLPort. On Fri, 24 Sep 2004 07:42:51 -0700 Reid Spencer <reid at x10sys.com> wrote: > Someone needs

Adding an include for std::remove under vc7.1 --- Paolo Invernizzi -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: diff.txt URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20040924/e1ca1218/attachment.txt>

On Wed, 15 Sep 2004 07:39:23 -0700 Reid Spencer <reid at x10sys.com> wrote: > Yes, in fact I'd be daring enough to suggest that it be the standard. > We'll have fewer compilation problems with VC++ 2005 because it is > (supposedly) more standards compliant than previous versions. Please use > this download: While this may be true, it's not a realistic request. Even

On Wed, 15 Sep 2004 19:51:08 +0200 Paolo Invernizzi <arathorn at fastwebnet.it> wrote: > > On Sep 15, 2004, at 7:29 PM, Jeff Cohen wrote: > > > Just make sure you read the uninstall instructions before installing. > > Microsoft has been known to essentially require the reformatting your > > boot partition in order to uninstall beta versions of VS. And, of >

<algorithm> works too. On Fri, 24 Sep 2004 10:09:21 -0500 Alkis Evlogimenos <alkis at cs.uiuc.edu> wrote: > On Fri, 2004-09-24 at 09:43, Paolo Invernizzi wrote: > > Jeff Cohen wrote: > > > > >But I compiled that under vc7.1 as it was! > > > > > > > > ;-(( > > > > Probably is an implicid includes, but I'm using the

But I compiled that under vc7.1 as it was! On Fri, 24 Sep 2004 15:19:22 +0200 Paolo Invernizzi <arathorn at fastwebnet.it> wrote: > Adding an include for std::remove under vc7.1 > > --- > Paolo Invernizzi >

On Wed, 15 Sep 2004 10:18:01 -0700 Reid Spencer <reid at x10sys.com> wrote: > Again, while I can sympathize, this problem doesn't affect LLVM because > we have no legacy of compiling on Win32. Well, these were reasons why it takes so long for newer versions to catch on. Obviously they don't apply to LLVM. Yet :) > > And don't even think of doing this near the end