[PATCH] D156427: [clang] Improve hermeticity of clang header tests.

2023-07-31 Thread Simi Pallipurath via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG3f75d38a4d4d: [clang] Improve hermeticity of clang header tests. (authored by simpal01). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D156427: [clang] Improve hermeticity of clang header tests.

2023-07-27 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 created this revision. Herald added a project: All. simpal01 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Specifying --sysroot prevents these header tests from being located in standard library directories. Repository: r

[PATCH] D155808: [clang][driver][NFC] Call IsARMBigEndain function only for isARM and isThumb.

2023-07-25 Thread Simi Pallipurath via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9dcd232f4afa: [clang][driver][NFC] Call IsARMBigEndain function only for isARM and isThumb. (authored by simpal01). Repository: rG LLVM Github Mon

[PATCH] D155808: [clang][driver][NFC] Call IsARMBigEndain function only for isARM and isThumb.

2023-07-24 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 added a comment. In D155808#4528543 , @MaskRay wrote: > Add NFC if this is NFC? Done Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155808/new/ https://reviews.llvm.org/D155808 __

[PATCH] D155808: [clang][driver] Missing the condition in IsARMBigEndain function.

2023-07-24 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 updated this revision to Diff 543484. simpal01 added a comment. Addressing review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155808/new/ https://reviews.llvm.org/D155808 Files: clang/lib/Driver/ToolChains/BareMetal.cpp cl

[PATCH] D155808: [clang][driver] Missing the condition in ISARMBigEndain function.

2023-07-20 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 created this revision. Herald added a subscriber: kristof.beyls. Herald added a project: All. simpal01 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. IsARMBIgEndian function returns true only if: 1. The triples are either

[PATCH] D154786: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal driver For Arm.

2023-07-19 Thread Simi Pallipurath via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc1873d84a331: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal… (authored by simpal01). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D154786: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal driver For Arm.

2023-07-18 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 updated this revision to Diff 541522. simpal01 added a comment. isArmBigEndian -> isARMBigEndian Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154786/new/ https://reviews.llvm.org/D154786 Files: clang/lib/Driver/ToolChains/Arch/ARM.cpp

[PATCH] D154786: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal driver For Arm.

2023-07-18 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 added a comment. @MaskRay [Gentle Reminder] I have implemented the requested changes. Could you please have a look again. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154786/new/ https://reviews.llvm.org/D154786 ___

[PATCH] D154786: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal driver For Arm.

2023-07-12 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 updated this revision to Diff 539582. simpal01 added a comment. Addressing Review comment - Removing the variable IsBigEndian. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154786/new/ https://reviews.llvm.org/D154786 Files: clang/lib/D

[PATCH] D154786: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal driver For Arm.

2023-07-11 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 updated this revision to Diff 539031. simpal01 added a comment. Addressing Review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154786/new/ https://reviews.llvm.org/D154786 Files: clang/lib/Driver/ToolChains/Arch/ARM.cpp cla

[PATCH] D154786: [Clang][Driver] Pass through the --be8 endian flag to linker in BareMetal driver For Arm.

2023-07-09 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 created this revision. Herald added subscribers: abidh, kristof.beyls. Herald added a project: All. simpal01 requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. When linking a big-endian image for Arm, clang has to select betw

[PATCH] D136957: [AArch64] Add support for the Cortex-A715 CPU

2022-11-03 Thread Simi Pallipurath via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGfa8aeab606c1: [AArch64] Add support for the Cortex-A715 CPU (authored by simpal01). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D136957: [AArch64] Add support for the Cortex-A715 CPU

2022-11-02 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 updated this revision to Diff 472560. simpal01 marked 2 inline comments as done. simpal01 added a comment. Comments addressed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136957/new/ https://reviews.llvm.org/D136957 Files: clang/docs/

[PATCH] D136957: [AArch64] Add support for the Cortex-A715 CPU

2022-10-31 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 updated this revision to Diff 471991. simpal01 added a comment. Comments Addressed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136957/new/ https://reviews.llvm.org/D136957 Files: clang/docs/ReleaseNotes.rst clang/test/Driver/aarch6

[PATCH] D136957: [AArch64] Add support for the Cortex-A715 CPU

2022-10-28 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 created this revision. simpal01 added reviewers: vhscampos, dmgreen, stuij. Herald added subscribers: hiraditya, kristof.beyls. Herald added a project: All. simpal01 requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits.

[PATCH] D65000: [ARM] Set default alignment to 64bits

2019-07-26 Thread Simi Pallipurath via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL367119: [ARM] Set default alignment to 64bits (authored by simpal01, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION

[PATCH] D42978: Make march/target-cpu print a note with the list of valid values for ARM

2018-03-15 Thread Simi Pallipurath via Phabricator via cfe-commits
simpal01 added a comment. @erichkeane Could you please tell me why this was done only for -cc1 and if there is any plan to add this to the driver in general ? Repository: rL LLVM https://reviews.llvm.org/D42978 ___ cfe-commits mailing list cfe-c