qiucf added a comment.

Thanks for the fix!



================
Comment at: clang/lib/Headers/ppc_wrappers/mm_malloc.h:22
 #else
+#if defined(__linux__)
 extern "C" int posix_memalign (void **, size_t, size_t) throw ();
----------------
Actually I just realised we don't need this guard, Clang already has a hack for 
this: previous declaration in system header has `throw ()` but later doesn't is 
okay.

See x86's header 
https://github.com/llvm/llvm-project/blob/510710d0374908e3e2bd37f9a32065da423a7e4c/clang/lib/Headers/mm_malloc.h#L21-L25
 .

I tested on Linux.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117972

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to