[PATCH] D130370: [llvm] Always use Constant for FP_ROUND ISD Nodes

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 22 11:08:26 PDT 2022

Previous message: [PATCH] D130370: [llvm] Always use Constant for FP_ROUND ISD Nodes
Next message: [PATCH] D130370: [llvm] Always use Constant for FP_ROUND ISD Nodes
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

craig.topper added a comment.

Why shouldn't it always be a TargetConstant? TargetConstant is what we use for ImmArg on intrinsics. This feels similar since we expect it to always be a constant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130370

Previous message: [PATCH] D130370: [llvm] Always use Constant for FP_ROUND ISD Nodes
Next message: [PATCH] D130370: [llvm] Always use Constant for FP_ROUND ISD Nodes
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list