[PATCH] D103562: [NFC][compiler-rt][hwasan] Refactor hwasan functions

[Florian Mayer via Phabricator via cfe-commits]

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