[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

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

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

https://github.com/SLTozer approved this pull request. It's been a bit without any comments, probably safe now to land this. https://github.com/llvm/llvm-project/pull/84756 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

OCHyams wrote: > Maybe wants someone with more frontend knowledge to chime in, but all SGTM. @adrian-prantl are you able to suggest anyone that might be able to take a look at this (part of this touches some Objective-C code+test) - we're not overly familiar with front end code. https://githu

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/84756 >From 8b37020a48b8145ad9d7deb288b9f8a3ee3a9a9b Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Sat, 9 Mar 2024 13:10:05 + Subject: [PATCH 1/3] clang: insertbefore non-debug for removeDIs stability

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

@@ -4746,20 +4746,32 @@ void CodeGenFunction::EmitOMPTaskBasedDirective( if (CGF.CGM.getCodeGenOpts().hasReducedDebugInfo()) (void)DI->EmitDeclareOfAutoVariable(SharedVar, ContextValue, CGF.Builder, false); -

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

https://github.com/SLTozer commented: Maybe wants someone with more frontend knowledge to chime in, but all SGTM. https://github.com/llvm/llvm-project/pull/84756 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mai

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

https://github.com/SLTozer edited https://github.com/llvm/llvm-project/pull/84756 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

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 e77f5fe889909df3508fd929f2636a0ac211877a efabe4b06f38aaf3aed8cf02c97f566652ba5f15 --

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Orlando Cazalet-Hyams (OCHyams) Changes This patch fixes problems that pop up when clang emits DbgRecords instead of debug intrinsics. Note: this doesn't mean clang is emitting DbgRecords yet, because the modules it creates are s

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Orlando Cazalet-Hyams (OCHyams) Changes This patch fixes problems that pop up when clang emits DbgRecords instead of debug intrinsics. Note: this doesn't mean clang is emitting DbgRecords yet, because the modules it creates are still alw

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

https://github.com/OCHyams edited https://github.com/llvm/llvm-project/pull/84756 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [RemoveDIs] Update Clang front end to handle DbgRecords (PR #84756)

https://github.com/OCHyams created https://github.com/llvm/llvm-project/pull/84756 This patch fixes problems that pop up when clang emits DbgRecords instead of debug intrinsics. Note: this doesn't mean clang is emitting DbgRecords yet, because the modules it creates are still always in the ol