RKSimon added a comment.
Please can you update the summary now that raptorlake + meteorlake are in the
same patch?
Comment at: clang/test/Preprocessor/predefined-arch-macros.c:2233
+// RUN: %clang -march=raptorlake -m32 -E -dM %s -o - 2>&1 \
+// RUN: -target i386-unknown
FreddyYe marked 3 inline comments as done.
FreddyYe added a comment.
THX for review! Gcc is also recently reviewing related patches. So to align
with the compiler-rt and libgcc, let's wait for their land first.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.l
FreddyYe updated this revision to Diff 468387.
FreddyYe added a comment.
Merge raptorlake patch and address comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135937/new/
https://reviews.llvm.org/D135937
Files:
clang/docs/ReleaseNotes.rst
RKSimon added a comment.
Please add this to llvm-project\llvm\test\CodeGen\X86\cpus-intel.ll
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135937/new/
https://reviews.llvm.org/D135937
___
cfe-commits mai
RKSimon added inline comments.
Comment at: llvm/lib/Target/X86/X86.td:1424
+def : ProcModel<"meteorlake", SLMModel, ProcessorFeatures.ADLFeatures,
+ProcessorFeatures.TRMTuning>;
This should be with the alderlake/raptorlake defs below (and use AD
craig.topper added a comment.
Release notes?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135937/new/
https://reviews.llvm.org/D135937
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
craig.topper added inline comments.
Comment at: clang/test/CodeGen/target-builtin-noerror.c:129
(void)__builtin_cpu_is("tremont");
+ (void)__builtin_cpu_is("meteorlake");
(void)__builtin_cpu_is("westmere");
I think this list was once in alphabetical order
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.
Repository:
rG LLVM Githu