http://llvm.org/bugs/show_bug.cgi?id=20590
David Binderman <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|WONTFIX |--- --- Comment #2 from David Binderman <[email protected]> --- >They're not initialized by itself, but with an argument of the same name. I don't think so. Please quote the relevant part of the C++ language standard that two mentions of variable S refer to two different objects. Let's not forget that this kind of double naming is poor software engineering. I'm thinking that a fix like SliceBuilder(const DataLayout &DL, AllocaInst &AI, AllocaSlices & thatS) : PtrUseVisitor<SliceBuilder>(DL), AllocSize(DL.getTypeAllocSize(AI.getAllocatedType())), S(thatS) {} would solve the problem. >This is a very common pattern in LLVM and harmless. Nope. I've just scanned all the trunk source code of LLVM and this problem pattern only occurs in this file. Hence the bug report. >What checker did emit those warnings? cppcheck, available from sourceforge. Strongly recommended. -- You are receiving this mail because: You are on the CC list for the bug.
_______________________________________________ LLVMbugs mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs
