llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-driver Author: Rainer Orth (rorth) <details> <summary>Changes</summary> `clang++` has long followed `g++`'s lead predefining `_XOPEN_SOURCE=600` on Solaris. As detailed in [Switch g++ to default Solaris 11.4 compilation environment](https://gcc.gnu.org/pipermail/gcc-patches/2026-May/716990.html), this is no longer necessary in Solaris 11.4. Tested on `amd64-pc-solaris2.11` and `sparcv9-sun-solaris2.11`. --- Full diff: https://github.com/llvm/llvm-project/pull/201063.diff 1 Files Affected: - (modified) clang/lib/Basic/Targets/OSTargets.h (-1) ``````````diff diff --git a/clang/lib/Basic/Targets/OSTargets.h b/clang/lib/Basic/Targets/OSTargets.h index 0fc387c90b280..943373c20af32 100644 --- a/clang/lib/Basic/Targets/OSTargets.h +++ b/clang/lib/Basic/Targets/OSTargets.h @@ -670,7 +670,6 @@ class LLVM_LIBRARY_VISIBILITY SolarisTargetInfo : public OSTargetInfo<Target> { DefineStd(Builder, "unix", Opts); Builder.defineMacro("__svr4__"); Builder.defineMacro("__SVR4"); - Builder.defineMacro("_XOPEN_SOURCE", "600"); if (Opts.CPlusPlus) { Builder.defineMacro("__C99FEATURES__"); Builder.defineMacro("_FILE_OFFSET_BITS", "64"); `````````` </details> https://github.com/llvm/llvm-project/pull/201063 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
