[PATCH] D94013: [libclc] Add clspv target for libclc

Jan Vesely via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 17 18:53:06 PST 2021


jvesely added a comment.

thanks. can you fix the "to avoid to use" part of the description? the "to avoid" part sounds redundant.
It explains why fma needs to be different, but it could use an explanation why it needs sw implementation of nextafter.

It still sounds to me like you can do `opt -O3 < builtins.opt.spirv-mesa3d-.bc > builtins.opt.clspv--.bc` and continue from there, with little to no difference.
The commit message should have enough information for anyone who would like to try that.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94013/new/

https://reviews.llvm.org/D94013



More information about the llvm-commits mailing list