craig.topper created this revision. craig.topper added reviewers: pengfei, RKSimon. Herald added a subscriber: StephenFan. Herald added a project: All. craig.topper requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang.
We use getHostCPUFeatures in x86::getX86TargetFeatures so I think this FIXME is taken care of already. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D137054 Files: clang/lib/Driver/ToolChains/Arch/X86.cpp Index: clang/lib/Driver/ToolChains/Arch/X86.cpp =================================================================== --- clang/lib/Driver/ToolChains/Arch/X86.cpp +++ clang/lib/Driver/ToolChains/Arch/X86.cpp @@ -30,9 +30,6 @@ // FIXME: Reject attempts to use -march=native unless the target matches // the host. - // - // FIXME: We should also incorporate the detected target features for use - // with -native. CPU = llvm::sys::getHostCPUName(); if (!CPU.empty() && CPU != "generic") return std::string(CPU);
Index: clang/lib/Driver/ToolChains/Arch/X86.cpp =================================================================== --- clang/lib/Driver/ToolChains/Arch/X86.cpp +++ clang/lib/Driver/ToolChains/Arch/X86.cpp @@ -30,9 +30,6 @@ // FIXME: Reject attempts to use -march=native unless the target matches // the host. - // - // FIXME: We should also incorporate the detected target features for use - // with -native. CPU = llvm::sys::getHostCPUName(); if (!CPU.empty() && CPU != "generic") return std::string(CPU);
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits