[PATCH] D112133: [DebugInfo][Instr] Track subregisters across stack spills/restores

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 21 04:19:35 PDT 2021


djtodoro added a comment.

In D112133#3077552 <https://reviews.llvm.org/D112133#3077552>, @jmorse wrote:

> In D112133#3077525 <https://reviews.llvm.org/D112133#3077525>, @djtodoro wrote:
>
>> I see... But I guess it can give us a decent location coverage improvement, especially for targets such as x86, right?
>
> Probably yes -- it's hard to say in advance because InstrRef is better at handling spill locations. It can follow values across both the stack and in registers, while VarLocBasedLDV has to commit to either following a spill or staying in a register. Definitely worth prototyping though.

Thanks.

> Note that I'm cooking up a plan to turn instr-ref on by default for X86 for LLVM14, although it's not a certain thing.

I am happy to hear that, good luck!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112133



More information about the llvm-commits mailing list