gamzinav/cling
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ValuePrinter: for auto decls, we should use the deduced type

Browse Source 
Value printing an expression of type `AutoType` ended up in calling the general
fallback `printValue(const void *)`.
To call the appropriate overload, the deduced type should be used instead.

Fixes ROOT-9687.
This commit is contained in:
Javier Lopez-Gomez 2022-02-22 14:39:26 +01:00 committed by jenkins
parent abe65b3bed
commit ba4ece8f3d
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/Interpreter/ValuePrinter.cpp
View File

@ -621,6 +621,15 @@ static const char* BuildAndEmitVPWrapperBody(cling::Interpreter &Interp,
R.begin(),
R.end());
// For `auto foo = bar;` decls, we are interested in the deduced type, i.e.
// AutoType 0x55e5ac848030 'int *' sugar
// `-PointerType 0x55e5ac847f70 'int *' << this type
// `-BuiltinType 0x55e5ab517420 'int'
if (auto AT = llvm::dyn_cast<clang::AutoType>(QT.getTypePtr())) {
if (AT->isDeduced())
QT = AT->getDeducedType();
}
if (auto PT = llvm::dyn_cast<clang::PointerType>(QT.getTypePtr())) {
// Normalize `X*` to `const void*`, invoke `printValue(const void**)`,
// unless it's a character string.