[libcxx-commits] [PATCH] D136257: [libc++abi][AIX] Use reserved slot in stack to pass the address of exception object

Xing Xue via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Oct 21 12:54:07 PDT 2022


xingxue marked an inline comment as done.
xingxue added inline comments.


================
Comment at: libcxxabi/src/aix_state_tab_eh.inc:664
+  // this function.
+  dummy();
+
----------------
cebowleratibm wrote:
> Suggestion: rename `dummy` to `force_a_stackframe` or similar.  The comments are helpful but the name makes it really hard for anyone to erroneously remove it, noting that you also have tests that would fail if anyone did but the more layers of defense and readability the better.
Renamed as suggested, thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136257



More information about the libcxx-commits mailing list