llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-backend-webassembly Author: Paulo Matos (pmatos) <details> <summary>Changes</summary> Enable address space map mangling for the WebAssembly target. This fixes a crash in the Itanium name mangler when trying to mangle types with the wasm_funcref address space qualifier in C++ mode. Fixes #<!-- -->176154 --- Full diff: https://github.com/llvm/llvm-project/pull/176237.diff 2 Files Affected: - (modified) clang/lib/Basic/Targets/WebAssembly.h (+1) - (modified) clang/test/CodeGenCXX/wasm-reftypes-mangle.cpp (+15-1) ``````````diff diff --git a/clang/lib/Basic/Targets/WebAssembly.h b/clang/lib/Basic/Targets/WebAssembly.h index 3634330ec6698..8cf5174300f82 100644 --- a/clang/lib/Basic/Targets/WebAssembly.h +++ b/clang/lib/Basic/Targets/WebAssembly.h @@ -81,6 +81,7 @@ class LLVM_LIBRARY_VISIBILITY WebAssemblyTargetInfo : public TargetInfo { explicit WebAssemblyTargetInfo(const llvm::Triple &T, const TargetOptions &) : TargetInfo(T) { AddrSpaceMap = &WebAssemblyAddrSpaceMap; + UseAddrSpaceMapMangling = true; NoAsmVariants = true; SuitableAlign = 128; LargeArrayMinWidth = 128; diff --git a/clang/test/CodeGenCXX/wasm-reftypes-mangle.cpp b/clang/test/CodeGenCXX/wasm-reftypes-mangle.cpp index c83176c3b7e4c..20e33cfdaa31c 100644 --- a/clang/test/CodeGenCXX/wasm-reftypes-mangle.cpp +++ b/clang/test/CodeGenCXX/wasm-reftypes-mangle.cpp @@ -2,5 +2,19 @@ // RUN: %clang_cc1 %s -triple wasm32-unknown-unknown -target-feature +reference-types -emit-llvm -o - -std=c++11 | FileCheck %s // RUN: %clang_cc1 %s -triple wasm64-unknown-unknown -target-feature +reference-types -emit-llvm -o - -std=c++11 | FileCheck %s -// CHECK: _Z2f1u11externref_t +// Test that funcref can be used in C++ without crashing during codegen. +// See https://github.com/llvm/llvm-project/issues/176154 +typedef void (*__funcref funcref_t)(); + +// Global funcref variables - test that codegen doesn't crash. +// CHECK-DAG: @fptr = global ptr addrspace(20) null +funcref_t fptr; + +// CHECK-DAG: @fpt2 = global ptr addrspace(20) null +void (*__funcref fpt2)(); + +// CHECK-DAG: _Z2f1u11externref_t void f1(__externref_t) {} + +// CHECK-DAG: _Z2f2PU4AS20FvvE +void f2(funcref_t) {} `````````` </details> https://github.com/llvm/llvm-project/pull/176237 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
