I think there are still some that only show in release builds. For instance<br><br>lib/CodeGen/MachineScheduler.cpp:356:12: error: private field 'NumInstrsScheduled' is not used [-Werror,-Wunused-private-field]<br>
unsigned NumInstrsScheduled;<br><br>Its only used by some NDEBUG code.<br><br><br>Also in lib/ExecutionEngine/JIT/JITEmitter.cpp. TheJIT is only used in asserts. I'm concerned that this implies that the MutexGuards here are only being used in debug builds and not release builds.<br>
<br> FunctionToLazyStubMapTy& getFunctionToLazyStubMap(<br> const MutexGuard& locked) {<br> assert(locked.holds(TheJIT->lock));<br> return FunctionToLazyStubMap;<br> }<br><br> GlobalToIndirectSymMapTy& getGlobalToIndirectSymMap(const MutexGuard& lck) {<br>
assert(lck.holds(TheJIT->lock));<br> return GlobalToIndirectSymMap;<br> }<br><br> std::pair<void *, Function *> LookupFunctionFromCallSite(<br> const MutexGuard &locked, void *CallSite) const {<br>
assert(locked.holds(TheJIT->lock));<br><br> // The address given to us for the stub may not be exactly right, it<br> // might be a little bit after the stub. As such, use upper_bound to<br> // find it.<br>
CallSiteToFunctionMapTy::const_iterator I =<br> CallSiteToFunctionMap.upper_bound(CallSite);<br> assert(I != CallSiteToFunctionMap.begin() &&<br> "This is not a known call site!");<br>
--I;<br> return *I;<br> }<br><br> void AddCallSite(const MutexGuard &locked, void *CallSite, Function *F) {<br> assert(locked.holds(TheJIT->lock));<br><br> bool Inserted = CallSiteToFunctionMap.insert(<br>
std::make_pair(CallSite, F)).second;<br> (void)Inserted;<br> assert(Inserted && "Pair was already in CallSiteToFunctionMap");<br> FunctionToCallSitesMap[F].insert(CallSite);<br> }<br>
<br><br><div class="gmail_quote">On Wed, Jun 6, 2012 at 12:47 PM, Benjamin Kramer <span dir="ltr"><<a href="mailto:benny.kra@googlemail.com" target="_blank">benny.kra@googlemail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: d0k<br>
Date: Wed Jun 6 14:47:08 2012<br>
New Revision: 158096<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=158096&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=158096&view=rev</a><br>
Log:<br>
Round 2 of dead private variable removal.<br>
<br>
LLVM is now -Wunused-private-field clean except for<br>
- lib/MC/MCDisassembler/Disassembler.h. Not sure why it keeps all those unaccessible fields.<br>
- gtest.<br>
<br>
Modified:<br>
llvm/trunk/include/llvm/Analysis/ProfileInfoLoader.h<br>
llvm/trunk/lib/Analysis/ProfileInfoLoader.cpp<br>
llvm/trunk/lib/Analysis/ProfileInfoLoaderPass.cpp<br>
llvm/trunk/lib/CodeGen/InlineSpiller.cpp<br>
llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp<br>
llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp<br>
llvm/trunk/lib/ExecutionEngine/MCJIT/MCJITMemoryManager.h<br>
llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.cpp<br>
llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.h<br>
llvm/trunk/lib/Target/CellSPU/SPUISelLowering.h<br>
llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.cpp<br>
llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.h<br>
llvm/trunk/lib/Target/Sparc/SparcFrameLowering.h<br>
llvm/trunk/lib/Target/XCore/XCoreFrameLowering.cpp<br>
llvm/trunk/lib/Target/XCore/XCoreFrameLowering.h<br>
llvm/trunk/tools/llvm-prof/llvm-prof.cpp<br>
llvm/trunk/unittests/Support/TypeBuilderTest.cpp<br>
llvm/trunk/utils/TableGen/DAGISelEmitter.h<br>
<br>
Modified: llvm/trunk/include/llvm/Analysis/ProfileInfoLoader.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/ProfileInfoLoader.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/ProfileInfoLoader.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/include/llvm/Analysis/ProfileInfoLoader.h (original)<br>
+++ llvm/trunk/include/llvm/Analysis/ProfileInfoLoader.h Wed Jun 6 14:47:08 2012<br>
@@ -28,7 +28,6 @@<br>
<br>
class ProfileInfoLoader {<br>
const std::string &Filename;<br>
- Module &M;<br>
std::vector<std::string> CommandLines;<br>
std::vector<unsigned> FunctionCounts;<br>
std::vector<unsigned> BlockCounts;<br>
@@ -39,8 +38,7 @@<br>
public:<br>
// ProfileInfoLoader ctor - Read the specified profiling data file, exiting<br>
// the program if the file is invalid or broken.<br>
- ProfileInfoLoader(const char *ToolName, const std::string &Filename,<br>
- Module &M);<br>
+ ProfileInfoLoader(const char *ToolName, const std::string &Filename);<br>
<br>
static const unsigned Uncounted;<br>
<br>
<br>
Modified: llvm/trunk/lib/Analysis/ProfileInfoLoader.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ProfileInfoLoader.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ProfileInfoLoader.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Analysis/ProfileInfoLoader.cpp (original)<br>
+++ llvm/trunk/lib/Analysis/ProfileInfoLoader.cpp Wed Jun 6 14:47:08 2012<br>
@@ -83,10 +83,8 @@<br>
// program if the file is invalid or broken.<br>
//<br>
ProfileInfoLoader::ProfileInfoLoader(const char *ToolName,<br>
- const std::string &Filename,<br>
- Module &TheModule) :<br>
- Filename(Filename),<br>
- M(TheModule), Warned(false) {<br>
+ const std::string &Filename)<br>
+ : Filename(Filename), Warned(false) {<br>
FILE *F = fopen(Filename.c_str(), "rb");<br>
if (F == 0) {<br>
errs() << ToolName << ": Error opening '" << Filename << "': ";<br>
<br>
Modified: llvm/trunk/lib/Analysis/ProfileInfoLoaderPass.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ProfileInfoLoaderPass.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ProfileInfoLoaderPass.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Analysis/ProfileInfoLoaderPass.cpp (original)<br>
+++ llvm/trunk/lib/Analysis/ProfileInfoLoaderPass.cpp Wed Jun 6 14:47:08 2012<br>
@@ -152,7 +152,7 @@<br>
}<br>
<br>
bool LoaderPass::runOnModule(Module &M) {<br>
- ProfileInfoLoader PIL("profile-loader", Filename, M);<br>
+ ProfileInfoLoader PIL("profile-loader", Filename);<br>
<br>
EdgeInformation.clear();<br>
std::vector<unsigned> Counters = PIL.getRawEdgeCounts();<br>
<br>
Modified: llvm/trunk/lib/CodeGen/InlineSpiller.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/InlineSpiller.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/InlineSpiller.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/CodeGen/InlineSpiller.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/InlineSpiller.cpp Wed Jun 6 14:47:08 2012<br>
@@ -52,7 +52,6 @@<br>
<br>
namespace {<br>
class InlineSpiller : public Spiller {<br>
- MachineFunctionPass &Pass;<br>
MachineFunction &MF;<br>
LiveIntervals &LIS;<br>
LiveStacks &LSS;<br>
@@ -137,8 +136,7 @@<br>
InlineSpiller(MachineFunctionPass &pass,<br>
MachineFunction &mf,<br>
VirtRegMap &vrm)<br>
- : Pass(pass),<br>
- MF(mf),<br>
+ : MF(mf),<br>
LIS(pass.getAnalysis<LiveIntervals>()),<br>
LSS(pass.getAnalysis<LiveStacks>()),<br>
AA(&pass.getAnalysis<AliasAnalysis>()),<br>
<br>
Modified: llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/RegAllocGreedy.cpp Wed Jun 6 14:47:08 2012<br>
@@ -73,7 +73,6 @@<br>
<br>
// analyses<br>
SlotIndexes *Indexes;<br>
- LiveStacks *LS;<br>
MachineDominatorTree *DomTree;<br>
MachineLoopInfo *Loops;<br>
EdgeBundles *Bundles;<br>
<br>
Modified: llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp (original)<br>
+++ llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp Wed Jun 6 14:47:08 2012<br>
@@ -45,7 +45,7 @@<br>
<br>
// If the target supports JIT code generation, create the JIT.<br>
if (TargetJITInfo *TJ = TM->getJITInfo())<br>
- return new MCJIT(M, TM, *TJ, new MCJITMemoryManager(JMM, M), GVsWithCode);<br>
+ return new MCJIT(M, TM, *TJ, new MCJITMemoryManager(JMM), GVsWithCode);<br>
<br>
if (ErrorStr)<br>
*ErrorStr = "target does not support JIT code generation";<br>
<br>
Modified: llvm/trunk/lib/ExecutionEngine/MCJIT/MCJITMemoryManager.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJITMemoryManager.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJITMemoryManager.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/ExecutionEngine/MCJIT/MCJITMemoryManager.h (original)<br>
+++ llvm/trunk/lib/ExecutionEngine/MCJIT/MCJITMemoryManager.h Wed Jun 6 14:47:08 2012<br>
@@ -22,15 +22,11 @@<br>
// matching LLVM IR counterparts in the module(s) being compiled.<br>
class MCJITMemoryManager : public RTDyldMemoryManager {<br>
virtual void anchor();<br>
- JITMemoryManager *JMM;<br>
+ OwningPtr<JITMemoryManager> JMM;<br>
<br>
- // FIXME: Multiple modules.<br>
- Module *M;<br>
public:<br>
- MCJITMemoryManager(JITMemoryManager *jmm, Module *m) :<br>
- JMM(jmm?jmm:JITMemoryManager::CreateDefaultMemManager()), M(m) {}<br>
- // We own the JMM, so make sure to delete it.<br>
- ~MCJITMemoryManager() { delete JMM; }<br>
+ MCJITMemoryManager(JITMemoryManager *jmm) :<br>
+ JMM(jmm?jmm:JITMemoryManager::CreateDefaultMemManager()) {}<br>
<br>
uint8_t *allocateDataSection(uintptr_t Size, unsigned Alignment,<br>
unsigned SectionID) {<br>
<br>
Modified: llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.cpp (original)<br>
+++ llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.cpp Wed Jun 6 14:47:08 2012<br>
@@ -30,12 +30,6 @@<br>
// very little right now.<br>
//===----------------------------------------------------------------------===//<br>
<br>
-SPUHazardRecognizer::SPUHazardRecognizer(const TargetInstrInfo &tii) :<br>
- TII(tii),<br>
- EvenOdd(0)<br>
-{<br>
-}<br>
-<br>
/// Return the pipeline hazard type encountered or generated by this<br>
/// instruction. Currently returns NoHazard.<br>
///<br>
<br>
Modified: llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.h (original)<br>
+++ llvm/trunk/lib/Target/CellSPU/SPUHazardRecognizers.h Wed Jun 6 14:47:08 2012<br>
@@ -24,12 +24,8 @@<br>
/// SPUHazardRecognizer<br>
class SPUHazardRecognizer : public ScheduleHazardRecognizer<br>
{<br>
-private:<br>
- const TargetInstrInfo &TII;<br>
- int EvenOdd;<br>
-<br>
public:<br>
- SPUHazardRecognizer(const TargetInstrInfo &TII);<br>
+ SPUHazardRecognizer(const TargetInstrInfo &/*TII*/) {}<br>
virtual HazardType getHazardType(SUnit *SU, int Stalls);<br>
virtual void EmitInstruction(SUnit *SU);<br>
virtual void AdvanceCycle();<br>
<br>
Modified: llvm/trunk/lib/Target/CellSPU/SPUISelLowering.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUISelLowering.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CellSPU/SPUISelLowering.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/CellSPU/SPUISelLowering.h (original)<br>
+++ llvm/trunk/lib/Target/CellSPU/SPUISelLowering.h Wed Jun 6 14:47:08 2012<br>
@@ -86,7 +86,6 @@<br>
class SPUTargetLowering :<br>
public TargetLowering<br>
{<br>
- int VarArgsFrameIndex; // FrameIndex for start of varargs area.<br>
SPUTargetMachine &SPUTM;<br>
<br>
public:<br>
<br>
Modified: llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.cpp (original)<br>
+++ llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.cpp Wed Jun 6 14:47:08 2012<br>
@@ -250,12 +250,8 @@<br>
<br>
NVPTXRegisterInfo::NVPTXRegisterInfo(const TargetInstrInfo &tii,<br>
const NVPTXSubtarget &st)<br>
-: NVPTXGenRegisterInfo(0),<br>
- TII(tii),<br>
- ST(st) {<br>
- Is64Bit = st.is64Bit();<br>
-}<br>
-<br>
+ : NVPTXGenRegisterInfo(0),<br>
+ Is64Bit(st.is64Bit()) {}<br>
<br>
#define GET_REGINFO_TARGET_DESC<br>
#include "NVPTXGenRegisterInfo.inc"<br>
<br>
Modified: llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.h (original)<br>
+++ llvm/trunk/lib/Target/NVPTX/NVPTXRegisterInfo.h Wed Jun 6 14:47:08 2012<br>
@@ -31,8 +31,6 @@<br>
<br>
class NVPTXRegisterInfo : public NVPTXGenRegisterInfo {<br>
private:<br>
- const TargetInstrInfo &TII;<br>
- const NVPTXSubtarget &ST;<br>
bool Is64Bit;<br>
// Hold Strings that can be free'd all together with NVPTXRegisterInfo<br>
ManagedStringPool ManagedStrPool;<br>
<br>
Modified: llvm/trunk/lib/Target/Sparc/SparcFrameLowering.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Sparc/SparcFrameLowering.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Sparc/SparcFrameLowering.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/Sparc/SparcFrameLowering.h (original)<br>
+++ llvm/trunk/lib/Target/Sparc/SparcFrameLowering.h Wed Jun 6 14:47:08 2012<br>
@@ -22,10 +22,9 @@<br>
class SparcSubtarget;<br>
<br>
class SparcFrameLowering : public TargetFrameLowering {<br>
- const SparcSubtarget &STI;<br>
public:<br>
- explicit SparcFrameLowering(const SparcSubtarget &sti)<br>
- : TargetFrameLowering(TargetFrameLowering::StackGrowsDown, 8, 0), STI(sti) {<br>
+ explicit SparcFrameLowering(const SparcSubtarget &/*sti*/)<br>
+ : TargetFrameLowering(TargetFrameLowering::StackGrowsDown, 8, 0) {<br>
}<br>
<br>
/// emitProlog/emitEpilog - These methods insert prolog and epilog code into<br>
<br>
Modified: llvm/trunk/lib/Target/XCore/XCoreFrameLowering.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/XCore/XCoreFrameLowering.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/XCore/XCoreFrameLowering.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/XCore/XCoreFrameLowering.cpp (original)<br>
+++ llvm/trunk/lib/Target/XCore/XCoreFrameLowering.cpp Wed Jun 6 14:47:08 2012<br>
@@ -78,8 +78,7 @@<br>
//===----------------------------------------------------------------------===//<br>
<br>
XCoreFrameLowering::XCoreFrameLowering(const XCoreSubtarget &sti)<br>
- : TargetFrameLowering(TargetFrameLowering::StackGrowsDown, 4, 0),<br>
- STI(sti) {<br>
+ : TargetFrameLowering(TargetFrameLowering::StackGrowsDown, 4, 0) {<br>
// Do nothing<br>
}<br>
<br>
<br>
Modified: llvm/trunk/lib/Target/XCore/XCoreFrameLowering.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/XCore/XCoreFrameLowering.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/XCore/XCoreFrameLowering.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/XCore/XCoreFrameLowering.h (original)<br>
+++ llvm/trunk/lib/Target/XCore/XCoreFrameLowering.h Wed Jun 6 14:47:08 2012<br>
@@ -22,7 +22,6 @@<br>
class XCoreSubtarget;<br>
<br>
class XCoreFrameLowering: public TargetFrameLowering {<br>
- const XCoreSubtarget &STI;<br>
public:<br>
XCoreFrameLowering(const XCoreSubtarget &STI);<br>
<br>
<br>
Modified: llvm/trunk/tools/llvm-prof/llvm-prof.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-prof/llvm-prof.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-prof/llvm-prof.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/tools/llvm-prof/llvm-prof.cpp (original)<br>
+++ llvm/trunk/tools/llvm-prof/llvm-prof.cpp Wed Jun 6 14:47:08 2012<br>
@@ -281,7 +281,7 @@<br>
// using the standard profile info provider pass, but for now this gives us<br>
// access to additional information not exposed via the ProfileInfo<br>
// interface.<br>
- ProfileInfoLoader PIL(argv[0], ProfileDataFile, *M);<br>
+ ProfileInfoLoader PIL(argv[0], ProfileDataFile);<br>
<br>
// Run the printer pass.<br>
PassManager PassMgr;<br>
<br>
Modified: llvm/trunk/unittests/Support/TypeBuilderTest.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Support/TypeBuilderTest.cpp?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Support/TypeBuilderTest.cpp?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/unittests/Support/TypeBuilderTest.cpp (original)<br>
+++ llvm/trunk/unittests/Support/TypeBuilderTest.cpp Wed Jun 6 14:47:08 2012<br>
@@ -167,13 +167,13 @@<br>
&(TypeBuilder<types::i<1>, true>::get(context2))->getContext());<br>
}<br>
<br>
-class MyType {<br>
+struct MyType {<br>
int a;<br>
int *b;<br>
void *array[1];<br>
};<br>
<br>
-class MyPortableType {<br>
+struct MyPortableType {<br>
int32_t a;<br>
int32_t *b;<br>
void *array[1];<br>
<br>
Modified: llvm/trunk/utils/TableGen/DAGISelEmitter.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/DAGISelEmitter.h?rev=158096&r1=158095&r2=158096&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/DAGISelEmitter.h?rev=158096&r1=158095&r2=158096&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/utils/TableGen/DAGISelEmitter.h (original)<br>
+++ llvm/trunk/utils/TableGen/DAGISelEmitter.h Wed Jun 6 14:47:08 2012<br>
@@ -23,10 +23,9 @@<br>
/// and emission of the instruction selector.<br>
///<br>
class DAGISelEmitter : public TableGenBackend {<br>
- RecordKeeper &Records;<br>
CodeGenDAGPatterns CGP;<br>
public:<br>
- explicit DAGISelEmitter(RecordKeeper &R) : Records(R), CGP(R) {}<br>
+ explicit DAGISelEmitter(RecordKeeper &R) : CGP(R) {}<br>
<br>
// run - Output the isel, returning true on failure.<br>
void run(raw_ostream &OS);<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>