jasonliu added inline comments.
================ Comment at: clang/include/clang/Basic/CodeGenOptions.def:47 +CODEGENOPT(DataSections , 1, 0) ///< Set by default, or when -f[no-]data-sections. +CODEGENOPT(HasExplicitDataSections, 1, 0) ///< Set when -f[no-]data-sections is set. CODEGENOPT(UniqueSectionNames, 1, 1) ///< Set for -funique-section-names. ---------------- MaskRay wrote: > From the current code. I don't see HasExplicitDataSections is necessary. > Please remove the lld changes. The reason I need `HasExplicitDataSections` goes back to D88493(which I haven't land yet, because I actually need to land these two patches together to avoid build break). In D88493, I'm trying to get the llc's default for -data-sections to be correct for AIX and introduced `HasExplicitDataSections`. If `HasExplicitDataSections` is not set, then llc would think there is -data-sections set, so it would just take the target triple's default, which makes `DataSections` setting to be useless. It seems there is no good way to set a certain TargetOption's default to be dependant on the current target triple. As I already mentioned in my own comment in this patch, one of the way to achieve that could be make DataSections an enum option in TargetOptions, so that it could have a `Default` enum which could mean true or false depending on the triple setting. But it could mean a bigger refactoring to this option. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88737/new/ https://reviews.llvm.org/D88737 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
