User Details
- User Since
- Jan 16 2018, 6:47 AM (157 w, 3 d)
Oct 13 2020
Oct 9 2020
Oct 1 2020
Sep 30 2020
It doesn't seem like anyone has any more immediate feedback. @efriedma, have all of your review comments been adequately addressed?
Sep 29 2020
LGTM, but @aaron.ballman should probably give his two cents as well for completion's sake.
Rebased.
Rebased.
Sep 25 2020
Ping. The build failures don't seem related to the patch.
Another gentle ping.
Sep 24 2020
So is this patch good to go then?
Sep 23 2020
Sep 22 2020
Sep 21 2020
Ping.
Ping. Any further comments?
Sep 15 2020
Any more feedback on this?
Fixed typos.
Sep 14 2020
Fix clang-format warnings.
Fix clang-format warnings.
Now only uses FP_TO_INT_SAT nodes for lowering. The nodes are selected in isel.
Sep 11 2020
Fix typo.
Using Type::getFloatingPointTy now.
Sep 10 2020
Use default expansion for f128 conversions; they aren't native so we can't be sure that they saturate.
Removed NativeSaturation parameter and made all lowering happen in Lower.
Sep 9 2020
Redid lowering.
Followup to removing vector expansion in the parent patch.
Removed vector expansion.
Fixed failing test cases.
Sep 8 2020
Updated method name.
Addressed comments.
Sep 7 2020
Removed superfluous NaN check and updated tests.
I don't know why the premerge tests are failing. I don't see those failures locally, so it's rather hard to debug it, especially since the premerge build lacks proper debug info.
Rebased.
Rebased.
Rebased.
Rebased.
Rebased.
Rebased.
Added promotion mechanism.
Added a promotion mechanism to handle cases where the floating point type cannot be used to rescale the value.
Sep 4 2020
Sep 3 2020
Sep 2 2020
Sep 1 2020
Made natively saturating expansions use integer comparisons instead.
Aug 31 2020
Addressed comments.
Aug 27 2020
I hadn't considered half precision as that's sort of off my radar. That does make both these conversion methods and the corresponding codegen implementation rather problematic.
Fix comment mistake.
This needs a few tests.
Aug 26 2020
Rebased and did minor adjustments.
Ping.
Aug 25 2020
Aug 24 2020
@gousemoodhin I just want to check; are you working on or planning to work on float-fixed support in the near future? Floating point conversions are the next step for the fixed-point support, so if you aren't immediately looking at it, I would like to take a shot at it.
Aug 21 2020
Added comments and more tests.
Revamped patch. It's now based on the FixedPointBuilder.
Aug 20 2020
Aug 19 2020
Amended comment.