mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-22 18:54:02 +01:00
29ffba4b56
This can be seen as a follow up to commit 0ee439b705e82a4fe20e2, that changed the second argument of __powidf2, __powisf2 and __powitf2 in compiler-rt from si_int to int. That was to align with how those runtimes are defined in libgcc. One thing that seem to have been missing in that patch was to make sure that the rest of LLVM also handle that the argument now depends on the size of int (not using the si_int machine mode for 32-bit). When using __builtin_powi for a target with 16-bit int clang crashed. And when emitting libcalls to those rtlib functions, typically when lowering @llvm.powi), the backend would always prepare the exponent argument as an i32 which caused miscompiles when the rtlib was compiled with 16-bit int. The solution used here is to use an overloaded type for the second argument in @llvm.powi. This way clang can use the "correct" type when lowering __builtin_powi, and then later when emitting the libcall it is assumed that the type used in @llvm.powi matches the rtlib function. One thing that needed some extra attention was that when vectorizing calls several passes did not support that several arguments could be overloaded in the intrinsics. This patch allows overload of a scalar operand by adding hasVectorInstrinsicOverloadedScalarOpd, with an entry for powi. Differential Revision: https://reviews.llvm.org/D99439 |
||
---|---|---|
.. | ||
basic-inseltpoison.ll | ||
basic.ll | ||
cache-bug.ll | ||
constant-extractelement.ll | ||
constant-insertelement.ll | ||
crash-bug.ll | ||
dbginfo.ll | ||
dbgloc-bug-inseltpoison.ll | ||
dbgloc-bug.ll | ||
global-bug-2.ll | ||
global-bug.ll | ||
intrinsics.ll | ||
order-bug-inseltpoison.ll | ||
order-bug.ll | ||
phi-bug-inseltpoison.ll | ||
phi-bug.ll | ||
phi-unreachable-pred.ll | ||
scatter-order.ll | ||
store-bug.ll | ||
variable-extractelement.ll | ||
variable-insertelement.ll | ||
vector-gep.ll |