diff --git a/lib/Target/CBackend/CBackend.cpp b/lib/Target/CBackend/CBackend.cpp index d778d1e946f..e32987cbd2e 100644 --- a/lib/Target/CBackend/CBackend.cpp +++ b/lib/Target/CBackend/CBackend.cpp @@ -163,11 +163,6 @@ namespace { }; } -// A pointer type should not use parens around *'s alone, e.g., (**) -inline bool ptrTypeNameNeedsParens(const std::string &NameSoFar) { - return NameSoFar.find_last_not_of('*') != std::string::npos; -} - // Pass the Type* and the variable name and this prints out the variable // declaration. // @@ -240,12 +235,8 @@ std::ostream &CWriter::printType(std::ostream &Out, const Type *Ty, const PointerType *PTy = cast(Ty); std::string ptrName = "*" + NameSoFar; - // Do not need parens around "* NameSoFar" if NameSoFar consists only - // of zero or more '*' chars *and* this is not an unnamed pointer type - // such as the result type in a cast statement. Otherwise, enclose in ( ). - if (ptrTypeNameNeedsParens(NameSoFar) || - PTy->getElementType()->getPrimitiveID() == Type::ArrayTyID) - ptrName = "(" + ptrName + ")"; // + if (isa(PTy->getElementType())) + ptrName = "(" + ptrName + ")"; return printType(Out, PTy->getElementType(), ptrName); } diff --git a/lib/Target/CBackend/Writer.cpp b/lib/Target/CBackend/Writer.cpp index d778d1e946f..e32987cbd2e 100644 --- a/lib/Target/CBackend/Writer.cpp +++ b/lib/Target/CBackend/Writer.cpp @@ -163,11 +163,6 @@ namespace { }; } -// A pointer type should not use parens around *'s alone, e.g., (**) -inline bool ptrTypeNameNeedsParens(const std::string &NameSoFar) { - return NameSoFar.find_last_not_of('*') != std::string::npos; -} - // Pass the Type* and the variable name and this prints out the variable // declaration. // @@ -240,12 +235,8 @@ std::ostream &CWriter::printType(std::ostream &Out, const Type *Ty, const PointerType *PTy = cast(Ty); std::string ptrName = "*" + NameSoFar; - // Do not need parens around "* NameSoFar" if NameSoFar consists only - // of zero or more '*' chars *and* this is not an unnamed pointer type - // such as the result type in a cast statement. Otherwise, enclose in ( ). - if (ptrTypeNameNeedsParens(NameSoFar) || - PTy->getElementType()->getPrimitiveID() == Type::ArrayTyID) - ptrName = "(" + ptrName + ")"; // + if (isa(PTy->getElementType())) + ptrName = "(" + ptrName + ")"; return printType(Out, PTy->getElementType(), ptrName); }