[PATCH] D158046: [X86] Support -march=gracemont

2023-08-20 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 rG6acff5390d05: [X86] Support -march=gracemont (authored by FreddyYe). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-18 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon accepted this revision. RKSimon added a comment. This revision is now accepted and ready to land. LGTM - cheers Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158046/new/ https://reviews.llvm.org/D158046

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-17 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe added inline comments. Comment at: clang/lib/Basic/Targets/X86.cpp:551 case CK_Lunarlake: + case CK_Gracemont: case CK_Sierraforest: RKSimon wrote: > FreddyYe wrote: > > RKSimon wrote: > > > Why not handle this above (below tremont) as the next

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-17 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe updated this revision to Diff 551350. FreddyYe marked 2 inline comments as done. FreddyYe added a comment. Address comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158046/new/ https://reviews.llvm.org/D158046 Files:

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-17 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added inline comments. Comment at: clang/lib/Basic/Targets/X86.cpp:551 case CK_Lunarlake: + case CK_Gracemont: case CK_Sierraforest: FreddyYe wrote: > RKSimon wrote: > > Why not handle this above (below tremont) as the next in the *mont series? >

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-17 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe marked an inline comment as done. FreddyYe added inline comments. Comment at: clang/lib/Basic/Targets/X86.cpp:551 case CK_Lunarlake: + case CK_Gracemont: case CK_Sierraforest: RKSimon wrote: > Why not handle this above (below tremont) as the next

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-17 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added inline comments. Comment at: clang/lib/Basic/Targets/X86.cpp:551 case CK_Lunarlake: + case CK_Gracemont: case CK_Sierraforest: Why not handle this above (below tremont) as the next in the *mont series? Repository: rG LLVM Github Monorepo

[PATCH] D158046: [X86] Support -march=gracemont

2023-08-15 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe created this revision. Herald added subscribers: Enna1, pengfei, hiraditya. Herald added a project: All. FreddyYe requested review of this revision. Herald added projects: clang, Sanitizers, LLVM. Herald added subscribers: llvm-commits, Sanitizers, cfe-commits. gracemont has some