craig.topper added inline comments.

================
Comment at: llvm/include/llvm/IR/IntrinsicsX86.td:2551
-              Intrinsic<[llvm_v4f32_ty], [llvm_v8i16_ty], [IntrNoMem]>;
-  def int_x86_vcvtph2ps_256 : GCCBuiltin<"__builtin_ia32_vcvtph2ps256">,
-              Intrinsic<[llvm_v8f32_ty], [llvm_v8i16_ty], [IntrNoMem]>;
----------------
hoyFB wrote:
> Hello, we have some tools generating these intrinsics being removed here. Can 
> you suggest what we should generate on LLVM IR with this change?
I think the diffs to test/CodeGen/f16c-builtins.c should show the difference. 
You can also follow the code in the changes to AutoUpgrade.cpp


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75162



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to