[PATCH] D124857: [AIX][PGO] Enable linux style PGO on AIX

wael yehia via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 3 09:45:52 PDT 2022


w2yehia added inline comments.


================
Comment at: compiler-rt/lib/profile/InstrProfilingPlatformLinux.c:11
+    (defined(__sun__) && defined(__svr4__)) || defined(__NetBSD__) || \
+    defined(_AIX)
 
----------------
jsji wrote:
> What is the major reason that we want to share the file with `Linux`? Would it be better to have AIX on its own file?
The main reason is the non-trivial amount of code duplication we will have if we split the AIX into its own file.


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

https://reviews.llvm.org/D124857



More information about the llvm-commits mailing list