[PATCH] D25261: [C API] Add LLVMConstBinOp function.

Amaury SECHET via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 14 13:21:18 PDT 2021

Previous message: [PATCH] D111831: [amdgpu] Fix a crash case when preserving MDT in SILowerControlFlow
Next message: [llvm] e9e6266 - [AArch64] Add extra tests for fptosisat vector variants
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

deadalnix added a comment.

Wouldn't `LLVMBuildBinOp` build a constant if both operand are constant and the operation allows for it? I have not double checked, but I think it does. Considering this, I'm not really sure what this brings to the table.


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

https://reviews.llvm.org/D25261

Previous message: [PATCH] D111831: [amdgpu] Fix a crash case when preserving MDT in SILowerControlFlow
Next message: [llvm] e9e6266 - [AArch64] Add extra tests for fptosisat vector variants
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list