This is an archive of the discontinued LLVM Phabricator instance.

[builtins] Support architectures with 16-bit int in __ashlti3, __ashrti3 and __lshrti3
ClosedPublic

Authored by Ka-Ka on Mar 27 2023, 6:58 AM.

Details

Summary

The amount to shift should be specified by the int type not a 32-bit integer
type.

This patch change the functions for 128-bit shifts in compiler-rt the same way
as was done for 64-bit shifts in D78662.

The README.txt is updated with the shift builtins signatures from this patch and D78662.

Diff Detail

Event Timeline

Ka-Ka created this revision.Mar 27 2023, 6:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2023, 6:58 AM
Herald added a subscriber: Enna1. · View Herald Transcript
Ka-Ka requested review of this revision.Mar 27 2023, 6:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2023, 6:58 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
bjope added a subscriber: bjope.Mar 28 2023, 1:03 AM
Ka-Ka updated this revision to Diff 509616.Mar 30 2023, 4:47 AM

git-clang-format

This revision is now accepted and ready to land.Mar 30 2023, 10:52 AM