[PATCH] D135488: [codegen] Display stack layouts in console

Tue Nov 15 13:02:06 PST 2022

arsenm added a comment.

In D135488#3928567 <https://reviews.llvm.org/D135488#3928567>, @nickdesaulniers wrote:

> In D135488#3928559 <https://reviews.llvm.org/D135488#3928559>, @arsenm wrote:
>
>> In D135488#3928557 <https://reviews.llvm.org/D135488#3928557>, @nickdesaulniers wrote:
>>
>>> Can we add a `Note` diagnostic when we emit a `-Wframe-larger-than=` that alludes to re-running with `-mllvm -print-stack-frame` to get additional info?
>>>
>>> We should update the release notes of clang to mention this feature, too.
>>
>> I don't think we should be pointing users to -mllvm flags. Plus, I don't really think random dbgs() printing is going to interact correctly with other diagnostics
>
> Then how will users know about super cool things? We could create a new front end flag for this `-mllvm` flag, a la https://reviews.llvm.org/D131986.

I'd be a bit more comfortable routing this through the backend remarks infrastructure, although it's a lot bigger than everything else currently reported there

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135488/new/

https://reviews.llvm.org/D135488