[libcxx-commits] [PATCH] D100132: [libunwind][AIX] implementation of the unwinder for AIX

[Fangrui Song via Phabricator via libcxx-commits]

MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

LGTM. Realized that you may need someone in the #libunwind <https://reviews.llvm.org/tag/libunwind/>

(Sorry for the belated response. I was mostly on trips and spent little time on catching up patches.)



================
Comment at: libunwind/src/assembly.h:212
+#define CSECT_ALIGN 3
+#elif defined(__ppc__)
+#define VBYTE_LEN 4
----------------
Can `#elif` be changed to `#else`?


================
Comment at: libunwind/src/config.h:63
 #else
-  #if !defined(__ELF__) && !defined(__MACH__)
-    #define _LIBUNWIND_EXPORT __declspec(dllexport)
-    #define _LIBUNWIND_HIDDEN
-  #else
-    #define _LIBUNWIND_EXPORT __attribute__((visibility("default")))
-    #define _LIBUNWIND_HIDDEN __attribute__((visibility("hidden")))
-  #endif
+#if !defined(__ELF__) && !defined(__MACH__) && !defined(_AIX)
+#define _LIBUNWIND_EXPORT __declspec(dllexport)
----------------
The re-indentation seems unneeded). You may just ignore the clang-foramt diagnostic.

If we want to do formatting, it should be a separate change; I believe the problem is that .clang-format does not precisely reflect the `PPIndentWidth` the code base actually uses.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100132