Quuxplusone added inline comments.
================ Comment at: clang/docs/ClangCommandLineReference.rst:2738 +Accepts any value for compatability reasons with GCC, thus not performing any CPU type specific tuning. + ---------------- "Accepts any value, for compatibility with GCC. Does not perform any CPU-specific tuning." In email I said: > there is no practical difference, but there may be a psychological > difference, between saying "Clang permanently treats -mtune as a no-op" > versus "Clang has a remarkably low //quality of implementation// for -mtune, > and has no immediate plans to either improve or regress it." To me, this patch's wording implies that Clang is promising to keep -mtune as a no-op forever. If Bob puts `-mtune=supercalifragilistic` in his Makefile today, and then in the future Clang starts emitting an "unknown target" error, will Bob point to this documentation as evidence that Clang promised to accept `-mtune=supercalifragilistic` as a no-op? So I would rather say: "Accepted for compatibility with GCC. Currently has no effect." CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78511/new/ https://reviews.llvm.org/D78511 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits