[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-05 Thread Freddy, Ye 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 rGdef720726b73: [X86][clang] Lift _BitInt() supported max width. (authored by FreddyYe). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-05 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe added a comment. THX for review. I'll land! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139170/new/ https://reviews.llvm.org/D139170 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-05 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe updated this revision to Diff 480305. FreddyYe added a comment. Rebase and add release note. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139170/new/ https://reviews.llvm.org/D139170 Files: clang/docs/ReleaseNotes.rst clang/lib/Basic

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-02 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. Thanks for working on this! LGTM, but please add a release note when landing so that users know about this improvement! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139170/new/ https://reviews.llvm.org/D139170

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-02 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe added a comment. In D139170#3965852 , @mgehre-amd wrote: > In D139170#3965850 , @FreddyYe > wrote: > >> In D139170#3965814 , @mgehre-amd >> wrote: >> >>> Do ot

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-02 Thread Matthias Gehre via Phabricator via cfe-commits
mgehre-amd added a comment. In D139170#3965850 , @FreddyYe wrote: > In D139170#3965814 , @mgehre-amd > wrote: > >> Do other targets not support > 128 bit integers, or is this PR only the >> first conservative st

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-02 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe added a comment. In D139170#3965814 , @mgehre-amd wrote: > Do other targets not support > 128 bit integers, or is this PR only the first > conservative step of lifting the limit? I temporary only enabled > 128 bit FP conversion for X86 in htt

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-01 Thread Matthias Gehre via Phabricator via cfe-commits
mgehre-amd added a comment. Do other targets not support > 128 bit integers, or is this PR only the first conservative step of lifting the limit? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139170/new/ https://reviews.llvm.org/D139170 _

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-01 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe created this revision. Herald added a project: All. FreddyYe requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D139170 Files: clang/lib/Basic/Targets/X86.h clang/te