Changes between Initial Version and Version 1 of Ticket #67802, comment 2


Ignore:
Timestamp:
Aug 2, 2023, 3:01:07 PM (14 months ago)
Author:
chrstphrchvz (Christopher Chavez)
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #67802, comment 2

    initial v1  
    1 Clang since 3.9.0 ([https://github.com/llvm/llvm-project/commit/9e7d0a98fa26]) but before 9.0.0 ([https://github.com/llvm/llvm-project/commit/6d9fb68c536d], and so likely also Xcode Clang before 11.4) had defined `_mm256_maskz_cvtps_ph` as 2-argument (and apparently so had Intel for a time, but not anymore). It should be okay to use the equivalent `_mm256_maskz_cvt_roundps_ph` intrinsic which Clang has always defined as 3-argument: [https://github.com/macports/macports-ports/pull/19627]
     1LLVM Clang since 3.9.0 ([https://github.com/llvm/llvm-project/commit/9e7d0a98fa26]) but before 9.0.0 ([https://github.com/llvm/llvm-project/commit/6d9fb68c536d], and so likely also Xcode Clang before 11.4) had defined `_mm256_maskz_cvtps_ph` as 2-argument (and apparently so had Intel for a time, but not anymore). It should be okay to use the equivalent `_mm256_maskz_cvt_roundps_ph` intrinsic which Clang has always defined as 3-argument: [https://github.com/macports/macports-ports/pull/19627]
    22
    33However [https://github.com/macports/macports-ports/pull/19407] already proposed disabling this and other code which does not build on 10.14.