Missed this check when porting from the libcall version.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM.
| lib/Transforms/InstCombine/InstCombineCasts.cpp | ||
|---|---|---|
| 1449 | It would help to have a comment here, maybe something like: // Except for fabs, this transformation requires the input of the unary FP operation to be itself an fpext from the type to which we're truncating. | |
It would help to have a comment here, maybe something like:
// Except for fabs, this transformation requires the input of the unary FP operation to be itself an fpext from the type to which we're truncating.