[PATCH] D92226: [VE] Change the behaviour of truncate

Simon Moll via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 30 02:28:57 PST 2020


simoll accepted this revision.
simoll added a comment.
This revision is now accepted and ready to land.

In D92226#2420812 <https://reviews.llvm.org/D92226#2420812>, @kaz7 wrote:

> Regarding to VE ABI.  VE ABI is compilicated.  It requires one-out for signed values and zero-out for unsigned values.  Therefore, I made LLVM for VE uses one-out before (I was thinking about signed values only at that time).  Recently, I modify clang for VE to pass all arguments in 64 bits and this fulfills VE's ABI.  Therefore, I decide to change LLVM for VE to use zero-out by default.

Fair enough, then.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92226/new/

https://reviews.llvm.org/D92226



More information about the llvm-commits mailing list