fmayer added inline comments.
================
Comment at: compiler-rt/lib/hwasan/hwasan.cpp:205
+ if (registers_frame && stack->trace && stack->size > 0) {
+ stack->trace++;
+ stack->size--;
----------------
fmayer wrote:
> vitalybuka wrote:
> > maybe we should pop everything up to "pc" to avoid issues with nested calls?
> >
> > For most users hwasan frames are not very useful. However if you work on
> > sanitizer, some frames can be a useful info. So I don't mind we just relax
> > test cases to accommodate this nesting.
> >
> > cc @smd
> This is probably for another patch though, right? This is already like this
> on the LHS.
nevermind. i accidentally had this left, sent https://reviews.llvm.org/D131279
for that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103562/new/
https://reviews.llvm.org/D103562
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
