Re: [OpenJDK 2D-Dev] RFR: 8265062: Remove duplication constant MaxTextureSize

2021-05-10 Thread Sergey Bylokhov 
On Thu, 15 Apr 2021 14:42:55 GMT, Denis Konoplev  wrote:

> I've removed MaxTextureSize and replaced its usages with 
> MTL_GPU_FAMILY_MAC_TXT_SIZE

Marked as reviewed by serb (Reviewer).

-

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

Re: [OpenJDK 2D-Dev] RFR: 8265062: Remove duplication constant MaxTextureSize

2021-05-06 Thread Phil Race 
On Thu, 15 Apr 2021 14:42:55 GMT, Denis Konoplev  wrote:

> I've removed MaxTextureSize and replaced its usages with 
> MTL_GPU_FAMILY_MAC_TXT_SIZE

Marked as reviewed by prr (Reviewer).

-

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

Re: [OpenJDK 2D-Dev] RFR: 8265062: Remove duplication constant MaxTextureSize

2021-05-04 Thread Denis Konoplev 
On Sat, 17 Apr 2021 04:34:06 GMT, Sergey Bylokhov  wrote:

> in what cases we can use the data larger than MTL_GPU_FAMILY_MAC_TXT_SIZE?

I think there are no such cases.

Sorry for slow reply, I was on vacation.

-

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

Re: [OpenJDK 2D-Dev] RFR: 8265062: Remove duplication constant MaxTextureSize

2021-04-16 Thread Sergey Bylokhov 
On Thu, 15 Apr 2021 14:42:55 GMT, Denis Konoplev  wrote:

> I've removed MaxTextureSize and replaced its usages with 
> MTL_GPU_FAMILY_MAC_TXT_SIZE

In a few places we have such pattern "sh = MIN(sh, 
MTL_GPU_FAMILY_MAC_TXT_SIZE);", in what cases we can use the data larger than 
MTL_GPU_FAMILY_MAC_TXT_SIZE since we should not create surfaces bigger than 
"MaxTextureSize"?

-

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

[OpenJDK 2D-Dev] RFR: 8265062: Remove duplication constant MaxTextureSize

2021-04-15 Thread Denis Konoplev 
I've removed MaxTextureSize and replaced its usages with 
MTL_GPU_FAMILY_MAC_TXT_SIZE

-

Commit messages:
 - 8265062: Remove duplication constant MaxTextureSize

Changes: https://git.openjdk.java.net/jdk/pull/3519/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk=3519=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8265062
  Stats: 10 lines in 3 files changed: 0 ins; 6 del; 4 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3519.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3519/head:pull/3519

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