RKSimon added inline comments.
================ Comment at: llvm/lib/Support/Host.cpp:828 + *Subtype = X86::INTEL_COREI7_RAPTORLAKE; + break; // Icelake Xeon: ---------------- newline to separate the defs ================ Comment at: llvm/lib/Target/X86/X86.td:1423 ProcessorFeatures.TRMTuning>; +def : ProcModel<"raptorlake", SLMModel, ProcessorFeatures.ADLFeatures, + ProcessorFeatures.TRMTuning>; ---------------- Shouldn't this be next to alderlake and use AlderlakePModel + ADLTuning? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135936/new/ https://reviews.llvm.org/D135936 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits