[libcxx-commits] [libcxx] [libcxxabi] [libc++][libc++abi] Minor follow-up changes after ptrauth upstreaming (PR #87481)

Daniil Kovalev via libcxx-commits libcxx-commits at lists.llvm.org
Mon Apr 8 05:42:17 PDT 2024


================
@@ -275,7 +275,7 @@ struct __type_info_implementations {
           __impl;
 };
 
-#    if defined(__arm64__) && __has_cpp_attribute(clang::ptrauth_vtable_pointer)
+#    if __has_cpp_attribute(clang::ptrauth_vtable_pointer)
 #      if __has_feature(ptrauth_type_info_discriminated_vtable_pointer)
----------------
kovdan01 wrote:

Let me put in my 2 cents: there is a link to ELF-related changes in https://github.com/llvm/llvm-project/issues/83805. Probably there was some discussion on the topic that I've missed, but in downstream we do not deleted check against `defined(__arm64__)` - instead, we change it to also accept `__aarch64__`: `(defined(__arm64__) || defined(__aarch64__))`. 

https://github.com/llvm/llvm-project/pull/87481


More information about the libcxx-commits mailing list