[PATCH] D107105: [AIX] Pass the -b option to linker on AIX (with fix to build break)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGaa35c496cf53: [AIX] Pass the -b option to linker on AIX (with fix to build break) (authored by anjankgk). Repository: rG LLVM Github Monorepo CHA

[PATCH] D107105: [AIX] Pass the -b option to linker on AIX (with fix to build break)

anjankgk created this revision. anjankgk added reviewers: ZarkoCA, cebowleratibm, hubert.reinterpretcast. Herald added a subscriber: dang. anjankgk requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch will re-enable the patch posted u

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG109954410c34: [AIX] Pass the -b option to linker on AIX (authored by anjankgk). Changed prior to commit: https://reviews.llvm.org/D106688?vs=36262

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk updated this revision to Diff 362622. anjankgk marked an inline comment as done. anjankgk added a comment. Change the error msg to include target information. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106688/new/ https://reviews.llvm.org/D106688 Files: clang/include/clan

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk marked an inline comment as done. anjankgk added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:263 + if (!T.isOSAIX()) { +TC.getDriver().Diag(diag::err_drv_unsupported_opt) +<< A.getAsString(Args); Zarko

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk added a comment. Thank you reviewing Zarko! Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:263 + if (!T.isOSAIX()) { +TC.getDriver().Diag(diag::err_drv_unsupported_opt) +<< A.getAsString(Args); ZarkoCA wrote: > nit, I p

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk marked an inline comment as done. anjankgk added inline comments. Comment at: clang/test/Driver/Xlinker-args.c:15-16 + +// RUN: %clang -target powerpc-unknown-aix -### \ +// RUN: -b one %s 2> %t +// RUN: FileCheck -check-prefix=AIX < %t %s ZarkoCA wrot

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk updated this revision to Diff 362412. anjankgk added a comment. Addressed review comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106688/new/ https://reviews.llvm.org/D106688 Files: clang/include/clang/Driver/Options.td clang/lib/Driver/ToolChains/CommonArgs.cpp cl

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk added a comment. Gentle ping.. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106688/new/ https://reviews.llvm.org/D106688 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.ll

[PATCH] D106688: [AIX] Pass the -b option to linker on AIX

anjankgk created this revision. anjankgk added reviewers: hubert.reinterpretcast, cebowleratibm, Whitney. Herald added a subscriber: dang. anjankgk requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. We want to parse the -b option in the driver

[PATCH] D106074: [AIX] Clang's library integration support for 128-bit long double is incomplete on AIX.

anjankgk added a comment. Since Whitny and Chris are busy, I have requested Shimin to commit this in. Thanks Shimin! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106074/new/ https://reviews.llvm.org/D106074 ___ cfe-commits mailing list cfe-c

[PATCH] D106371: [AIX] Generate large code model relocations when mcmodel=medium on AIX

anjankgk added a comment. Since Hubert is busy, I have requested Shimin to commit this in. Thanks Shimin! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106371/new/ https://reviews.llvm.org/D106371 ___ cfe-commits mailing list cfe-commits@list

[PATCH] D106371: [AIX] Generate large code model relocations when mcmodel=medium on AIX

anjankgk added a comment. Anjan Kumar Guttahalli Krishna Thank you! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106371/new/ https://reviews.llvm.org/D106371 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cg

[PATCH] D106074: [AIX] Clang's library integration support for 128-bit long double is incomplete on AIX.

anjankgk added a comment. In D106074#2893249 , @xgupta wrote: > @anjankgk You need to provide your identity for commit in "your name email>" form. > see https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access > >> Prior to obtaining commit acc

[PATCH] D106074: [AIX] Clang's library integration support for 128-bit long double is incomplete on AIX.

anjankgk added a comment. @cebowleratibm @Whitney Could either of you pls commit this? Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106074/new/ https://reviews.llvm.org/D106074 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D106371: [AIX] Generate large code model relocations when mcmodel=medium on AIX

anjankgk added a comment. In D106371#2892186 , @hubert.reinterpretcast wrote: > LGTM with minor nit; thanks. Done. @hubert.reinterpretcast Hubert, could you pls help me commit this? Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106371

[PATCH] D106371: [AIX] Generate large code model relocations when mcmodel=medium on AIX

anjankgk updated this revision to Diff 360443. anjankgk added a comment. Added braces for else-block. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106371/new/ https://reviews.llvm.org/D106371 Files: clang/lib/Driver/ToolChains/Clang.cpp clang/test/Driver/mcmodel.c Index: clang/t

[PATCH] D106371: [AIX] Generate large code model relocations when mcmodel=medium on AIX

anjankgk created this revision. anjankgk added reviewers: hubert.reinterpretcast, cebowleratibm, Whitney. anjankgk requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This patch makes the changes in the driver that converts the medium code mode

[PATCH] D106074: [AIX] Clang's library integration support for 128-bit long double is incomplete on AIX.

anjankgk updated this revision to Diff 359926. anjankgk retitled this revision from "[AIX] Emit unsupported 128-bit long double option for AIX" to "[AIX] Clang's library integration support for 128-bit long double is incomplete on AIX.". anjankgk edited the summary of this revision. anjankgk adde

[PATCH] D106074: [AIX] Emit unsupported 128-bit long double option for AIX

anjankgk added a comment. Thanks for reviewing! Can someone with commit access help commit this pls? Thanks in advance! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106074/new/ https://reviews.llvm.org/D106074

[PATCH] D106074: [AIX] Emit unsupported 128-bit long double option for AIX

anjankgk created this revision. anjankgk added reviewers: daltenty, Whitney, hubert.reinterpretcast. anjankgk requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. AIX currently doesn't support the 128-bit long double. Repository: rG LLVM Git