RFR: 8276905: Function frag_col has a deployment target which is incompatible with this OS [v2]

[Phil Race]

On Fri, 12 Nov 2021 03:56:57 GMT, Phil Race <prr at openjdk.org> wrote:

>> Jayathirth D V has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use Macro for version
>
> make/modules/java.desktop/lib/Awt2dLibraries.gmk line 850:
> 
>> 848:       COMMAND := $(METAL) -c -std=osx-metal2.0 \
>> 849:           -mmacosx-version-min=$(MACOSX_VERSION_MIN) \
>> 850:           -o $(SHADERS_AIR) $(SHADERS_SRC), \
> 
> No .. we decided metal requires macos 10.14 as a minimum. In fact it won't run (by policy) on earlier releases so settting it to what the broader JDK defines as a minimum is not appropriate right now.
> And I've no idea what restrictions we'd be placing on metal saying it can only use 10.12 features.
> Nor do I know what the xcode 12.4 used to build JDK 17 defaults to and how much a change to either 10.12 or 10.14 might require in re-testing.
> 
> So - 
> we should use 10.14 
> what's the actual impact of that on a current build using xcode 12.4 - does it change anything we use ?

bear in mind "impact" might mean metal can't use some new faster code, not just runtime errors

-------------

PR: https://git.openjdk.java.net/jdk/pull/6346