[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-18 Thread Michael Francis via Phabricator via cfe-commits
francii updated this revision to Diff 476649. francii added a comment. Remove Linux changes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137753/new/ https://reviews.llvm.org/D137753 Files: clang/include/clang/Basic/CodeGenOptions.def clang/in

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-18 Thread Michael Francis via Phabricator via cfe-commits
francii added a comment. In D137753#3935391 , @MaskRay wrote: > In D137753#3935305 , @francii wrote: > >> Recall that the goal with `-p` is to create parity with GCC (at least with >> Linux and AIX), as per the R

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. In D137753#3935617 , @francii wrote: > In D137753#3935391 , @MaskRay wrote: > >> In D137753#3935305 , @francii >> wrote: >> >>> Recall that the g

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Michael Francis via Phabricator via cfe-commits
francii added a comment. In D137753#3935391 , @MaskRay wrote: > In D137753#3935305 , @francii wrote: > >> Recall that the goal with `-p` is to create parity with GCC (at least with >> Linux and AIX), as per the R

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. In D137753#3935305 , @francii wrote: > Recall that the goal with `-p` is to create parity with GCC (at least with > Linux and AIX), as per the RFC discussion. > > In D137753#3935138 , @Ma

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Michael Francis via Phabricator via cfe-commits
francii added a comment. Recall that the goal with `-p` is to create parity with GCC (at least with Linux and AIX), as per the RFC discussion. In D137753#3935138 , @MaskRay wrote: > In D137753#3935126 , @francii

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. In D137753#3935126 , @francii wrote: > In D137753#3934932 , @MaskRay wrote: > >> Please make `-p` accepted for AIX only and don't change the semantics for >> other targets in this patch.

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Michael Francis via Phabricator via cfe-commits
francii added a comment. In D137753#3934932 , @MaskRay wrote: > Please make `-p` accepted for AIX only and don't change the semantics for > other targets in this patch. For FreeBSD and Linux (musl and gnu) we can try > rejecting `-p`. If OpenBSD wants t

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. Please make `-p` accepted for AIX only and don't change the semantics for other targets. For FreeBSD and Linux (musl and gnu) we can try rejecting `-p`. If OpenBSD wants to make `-p` an alias for `-pg`, that's fine. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-17 Thread Michael Francis via Phabricator via cfe-commits
francii updated this revision to Diff 476203. francii added a comment. Specify prof and gprof in help text Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137753/new/ https://reviews.llvm.org/D137753 Files: clang/include/clang/Basic/CodeGenOptions

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-14 Thread Michael Francis via Phabricator via cfe-commits
francii updated this revision to Diff 475242. francii added a comment. Add sysroot check to aix-ld.c test cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137753/new/ https://reviews.llvm.org/D137753 Files: clang/include/clang/Basic/CodeGenOp

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-14 Thread Michael Francis via Phabricator via cfe-commits
francii updated this revision to Diff 475196. francii added a comment. Add profiled libraries check AIX Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137753/new/ https://reviews.llvm.org/D137753 Files: clang/include/clang/Basic/CodeGenOptions.de

[PATCH] D137753: [Clang][GNU][AIX][p]Enable -p Functionality

2022-11-13 Thread Michael Francis via Phabricator via cfe-commits
francii updated this revision to Diff 475009. francii added a comment. Updated Linux test case Added profiled library search for AIX. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137753/new/ https://reviews.llvm.org/D137753 Files: clang/include

[PATCH] D137753: [Clang][GNU][AIX}[p]Enable -p Functionality

2022-11-10 Thread Michael Francis via Phabricator via cfe-commits
francii updated this revision to Diff 474604. francii added a comment. Added Linux implementation and test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137753/new/ https://reviews.llvm.org/D137753 Files: clang/include/clang/Basic/CodeGenOp