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
https://revi
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
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 in
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:
clang/lib/Basi
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?
> G
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
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
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 differen