[PATCH] D158169: [X86] Fix i128 argument passing under SysV ABI

2023-09-14 Thread Nikita Popov via Phabricator via cfe-commits
nikic added a comment. In D158169#4645012 , @RalfJung wrote: >> I think the CCIfSplit means it was larger than i64 to start. > > What about the case where `R9` would take the *second* half of a split i128 > (i.e., the value gets split across `R8` and

[PATCH] D158169: [X86] Fix i128 argument passing under SysV ABI

2023-09-13 Thread Ralf via Phabricator via cfe-commits
RalfJung added a comment. > I think the CCIfSplit means it was larger than i64 to start. What about the case where R9 would take the *second* half of a split i128 (i.e., the value gets split across R8 and R9

[PATCH] D158169: [X86] Fix i128 argument passing under SysV ABI

2023-08-21 Thread Nikita Popov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGfa1b6e6b34eb: [X86] Fix i128 argument passing under SysV ABI (authored by nikic). Herald added a project: clang. Herald added a subscriber: cfe-commits. Changed prior to commit: