[PATCH] D58042: [LiveDebugValues] Emit parameter's entry value
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 28 09:33:11 PDT 2019
aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.
LGTM with an extra assertion
================
Comment at: lib/CodeGen/LiveDebugValues.cpp:169
+ if (Kind != EntryValueKind)
+ Kind = RegisterKind;
Loc.RegNo = RegNo;
----------------
This looks dangerous.. are entry values somehow implicitly also register values? Iguess besed on the other patches the answer is yes, but that also means that we must `assert((!EntryValueKind || RegNo) && "entry values must be register locations")`
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58042/new/
https://reviews.llvm.org/D58042
More information about the llvm-commits
mailing list