[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

This revision was automatically updated to reflect the committed changes. Closed by commit rG4498663f3de0: [AST] Initialized data after TypeSourceInfo (authored by vitalybuka). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150499/new/

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

vitalybuka added reviewers: kstoimenov, thurston. vitalybuka added a comment. Oh, I rerun msan bot locally and it looked like issue dissipated, but I probably checked wrong logs. So I enabled -Oz on the bot, to catch more bugs, but the issue is still there

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

vitalybuka added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150499/new/ https://reviews.llvm.org/D150499 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

vitalybuka added a comment. ping, any concerns regarding this patch? i want to make MSAN stricter on our bot by reducing -O level Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150499/new/ https://reviews.llvm.org/D150499

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

vitalybuka added a comment. In D150499#4341115 , @rjmccall wrote: > This memory is supposed to be initialized when we copy the `TypeLoc`. Are > you observing that not happening? Yes. There is the msan report in the description. Repository: rG LLVM

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

rjmccall added a comment. This memory is supposed to be initialized when we copy the `TypeLoc`. Are you observing that not happening? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150499/new/ https://reviews.llvm.org/D150499

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

vitalybuka updated this revision to Diff 521847. vitalybuka added a comment. use 0 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150499/new/ https://reviews.llvm.org/D150499 Files: clang/include/clang/AST/Type.h

[PATCH] D150499: [AST] Initialized data after TypeSourceInfo

vitalybuka created this revision. Herald added a project: All. vitalybuka requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. There is no initialization of the data between allocation and first getBeginLoc call.