[llvm] Revert "[NFC] Don't recompute type name" (PR #119907)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 13 09:40:52 PST 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-support
Author: Ian Wood (IanWood1)
<details>
<summary>Changes</summary>
Breaks windows builds because `StringRef Key = "getTypeName<";` wasn't updated to `getTypeNameImpl`'s name
Reverts llvm/llvm-project#<!-- -->119631
---
Full diff: https://github.com/llvm/llvm-project/pull/119907.diff
1 Files Affected:
- (modified) llvm/include/llvm/Support/TypeName.h (+12-18)
``````````diff
diff --git a/llvm/include/llvm/Support/TypeName.h b/llvm/include/llvm/Support/TypeName.h
index 61ba09c2163047..9547e76a7fa79b 100644
--- a/llvm/include/llvm/Support/TypeName.h
+++ b/llvm/include/llvm/Support/TypeName.h
@@ -13,8 +13,18 @@
namespace llvm {
-namespace detail {
-template <typename DesiredTypeName> inline StringRef getTypeNameImpl() {
+/// We provide a function which tries to compute the (demangled) name of a type
+/// statically.
+///
+/// This routine may fail on some platforms or for particularly unusual types.
+/// Do not use it for anything other than logging and debugging aids. It isn't
+/// portable or dependendable in any real sense.
+///
+/// The returned StringRef will point into a static storage duration string.
+/// However, it may not be null terminated and may be some strangely aligned
+/// inner substring of a larger string.
+template <typename DesiredTypeName>
+inline StringRef getTypeName() {
#if defined(__clang__) || defined(__GNUC__)
StringRef Name = __PRETTY_FUNCTION__;
@@ -48,22 +58,6 @@ template <typename DesiredTypeName> inline StringRef getTypeNameImpl() {
return "UNKNOWN_TYPE";
#endif
}
-} // namespace detail
-
-/// We provide a function which tries to compute the (demangled) name of a type
-/// statically.
-///
-/// This routine may fail on some platforms or for particularly unusual types.
-/// Do not use it for anything other than logging and debugging aids. It isn't
-/// portable or dependendable in any real sense.
-///
-/// The returned StringRef will point into a static storage duration string.
-/// However, it may not be null terminated and may be some strangely aligned
-/// inner substring of a larger string.
-template <typename DesiredTypeName> inline StringRef getTypeName() {
- static StringRef Name = detail::getTypeNameImpl<DesiredTypeName>();
- return Name;
-}
} // namespace llvm
``````````
</details>
https://github.com/llvm/llvm-project/pull/119907
More information about the llvm-commits
mailing list