[PATCH] D23198: clang-rename rename-all: support reading old/newname pairs from a YAML file

Miklos Vajna via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 9 08:12:57 PDT 2016

vmiklos added inline comments.

Comment at: clang-rename/tool/ClangRename.cpp:65
@@ +64,3 @@
+  RenameAllInfo() : Offset(0) {}
omtcyfz wrote:
> AFAIK there's no need to do that, integer types are by default initialized with 0, aren't they?
Are you sure? Here is a minimal version that shows what goes wrong when that's not initialized explicitly: http://pastebin.com/raw/2ZsUgWf6 The "Use of uninitialised value of size 8" goes away with an explicit initialization.


More information about the cfe-commits mailing list