search for: machinefunctionprinterpass

...e svn version 97812 (the one the patch is made against), but it doesn't compile: "llvm/include/llvm/Pass.h:127: Error: expected unqualified-id before "&" token" Seems raw_ostream is forward declared but not defined (adding a missing #include fixed that). But the file MachineFunctionPrinterPass.h is still missing. kalle

...d-id before "&" > token" > Seems raw_ostream is forward declared but not defined (adding a missing > #include fixed that). It shouldn't need to be defined. Perhaps std::string is the issue, though I don't know why I don't see the problem. Probably because MachineFunctionPrinterPass.h includes <string> somewhere along the line. > But the file MachineFunctionPrinterPass.h is still missing. Thanks, here's an updated version. -Dave -------------- next part -------------- A non-text attachment was scrubbed... Name: llvm-...

...t;> token" >> Seems raw_ostream is forward declared but not defined (adding a missing >> #include fixed that). > > It shouldn't need to be defined. Perhaps std::string is the issue, though I > don't know why I don't see the problem. Probably because > MachineFunctionPrinterPass.h includes <string> somewhere along > the line. This is much better than the first iteration but still has many issues. There is no need to keep cc'ing cfe-dev on these patches which have nothing to do with clang. Please rename the getPrinterPass method to createPrinterPass to indic...

...in this set. > Please rename the getPrinterPass method to createPrinterPass to indicate > that the result returns a new pass, not returning an existing one. Please Ok. > make it take a StringRef, not an std::string and eliminate the <string> > #include from Pass.h. Ok. > MachineFunctionPrinterPass.h doesn't need to #include > MachineFunctionPass.h, just use a forward declaration. Also, please move Ok. > the prototype of createMachineFunctionPrinterPass into > include/llvm/CodeGen/Passes.h, eliminating the need for the header in the > first place. Ok. > This hunk looks...

...39;Machine' into the new library. These are: - MachineBasicBlock - MachineBlockFrequencyInfo - MachineBlockPlacement - MachineBranchProbabilityInfo - MachineCodeEmitter - MachineCopyPropagation - MachineCSE - MachineDominators - MachineFunctionAnalysis - MachineFunction - MachineFunctionPass - MachineFunctionPrinterPass - MachineInstrBundle - MachineInstr - MachineLICM - MachineLoopInfo - MachineModuleInfo - MachineModuleInfoImpls - MachinePassRegistry - MachinePostDominators - MachineRegisterInfo - MachineScheduler - MachineSink - MachineSSAUpdater - MachineTraceMetrics - MachineVerifier Are there any files in t...

...hineBasicBlock > - MachineBlockFrequencyInfo > - MachineBlockPlacement > - MachineBranchProbabilityInfo > - MachineCodeEmitter > - MachineCopyPropagation > - MachineCSE > - MachineDominators > - MachineFunctionAnalysis > - MachineFunction > - MachineFunctionPass > - MachineFunctionPrinterPass > - MachineInstrBundle > - MachineInstr > - MachineLICM > - MachineLoopInfo > - MachineModuleInfo > - MachineModuleInfoImpls > - MachinePassRegistry > - MachinePostDominators > - MachineRegisterInfo > - MachineScheduler > - MachineSink > - MachineSSAUpdater >...

On Thursday 04 March 2010 09:28:22 David Greene wrote: > I'm happy to revisit design decisions if you let me know what you want to > see changed. Here's a rework using PassManager as Chris suggested. Comments? -Dave -------------- next part -------------- A non-text attachment was scrubbed... Name: clang-beforeafter.patch Type:

...you probably don’t want. Pruning the list to basic IR support: > >> - MachineBasicBlock >> - MachineBranchProbabilityInfo >> - MachineCodeEmitter >> - MachineDominators >> - MachineFunctionAnalysis >> - MachineFunction >> - MachineFunctionPass >> - MachineFunctionPrinterPass >> - MachineInstrBundle >> - MachineInstr >> - MachineLoopInfo >> - MachineModuleInfo >> - MachineModuleInfoImpls >> - MachinePassRegistry >> - MachinePostDominators >> - MachineRegisterInfo >> - MachineSSAUpdater >> - MachineVerifier Tha...

On Thursday 04 March 2010 01:54:55 Chris Lattner wrote: > On Mar 2, 2010, at 1:45 PM, David Greene wrote: > > This set of patches adds support for dumping IR before or after specified > > Passes. It adds the following command-line options: > > > > -print-before=<pass-option> > > -print-after=<pass-option> > > -print-before-all > >

...es' llvm[2]: Compiling MachineFunctionPass.cpp for Release+Asserts build llvm[2]: Compiling SubtargetFeature.cpp for Release+Asserts build llvm[2]: Compiling WinCOFFObjectWriter.cpp for Release+Asserts build llvm[2]: Building Release+Asserts Archive Library libLLVMLibDriver.a llvm[2]: Compiling MachineFunctionPrinterPass.cpp for Release+Asserts build make[2]: Leaving directory '/home/NIKHILREDDY/WORK/LLVM_OBJ/lib/LibDriver' llvm[2]: Compiling WinCOFFStreamer.cpp for Release+Asserts build llvm[2]: Compiling YAML.cpp for Release+Asserts build llvm[2]: Compiling MachineInstr.cpp for Release+Asserts build llvm[...