[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/86852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/86852 >From 9ddf01a4f28df19914aa393b1ac518410693af5b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 14:33:15 -0400 Subject: [PATCH 1/4] [InstallAPI] Add support for parsing dSYMs InstallAPI doe

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

@@ -0,0 +1,40 @@ +; RUN: rm -rf %t +; RUN: split-file %s %t + +// Build a simple dylib with debug info. +; RUN: %clang --target=arm64-apple-macos13 -g -dynamiclib %t/foo.c \ cyndyishida wrote: Looks like the linker on the linux bot doesn't support building Darwin

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

https://github.com/ributzka approved this pull request. https://github.com/llvm/llvm-project/pull/86852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

@@ -511,14 +520,16 @@ DylibVerifier::Result DylibVerifier::verify(GlobalRecord *R, return verifyImpl(R, SymCtx); } -void DylibVerifier::VerifierContext::emitDiag( -llvm::function_ref Report) { +void DylibVerifier::VerifierContext::emitDiag(llvm::function_ref Report, +

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

@@ -511,14 +520,16 @@ DylibVerifier::Result DylibVerifier::verify(GlobalRecord *R, return verifyImpl(R, SymCtx); } -void DylibVerifier::VerifierContext::emitDiag( -llvm::function_ref Report) { +void DylibVerifier::VerifierContext::emitDiag(llvm::function_ref Report, +

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/86852 >From 9ddf01a4f28df19914aa393b1ac518410693af5b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 14:33:15 -0400 Subject: [PATCH 1/3] [InstallAPI] Add support for parsing dSYMs InstallAPI doe

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/86852 >From 9ddf01a4f28df19914aa393b1ac518410693af5b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 14:33:15 -0400 Subject: [PATCH 1/2] [InstallAPI] Add support for parsing dSYMs InstallAPI doe

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff dcd0f2b6103072b74b446c2d1e9ecec60001a28c 9ddf01a4f28df19914aa393b1ac518410693af5b --

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

@@ -0,0 +1,40 @@ +; RUN: rm -rf %t +; RUN: split-file %s %t + +// Build a simple dylib with debug info. +; RUN: %clang --target=arm64-apple-macos13 -g -dynamiclib %t/foo.c \ cyndyishida wrote: @JDevlieghere Is there a better/known practice for generating full dyl

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Cyndy Ishida (cyndyishida) Changes InstallAPI does not directly look at object files in the dylib for verification. To help diagnose violations where a declaration is undiscovered in headers, parse the dSYM and look up the source location

[clang] [llvm] [InstallAPI] Add support for parsing dSYMs (PR #86852)

https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/86852 InstallAPI does not directly look at object files in the dylib for verification. To help diagnose violations where a declaration is undiscovered in headers, parse the dSYM and look up the source location for