[llvm-branch-commits] [llvm] release/18.x: [llvm][LoongArch] Improve loongarch_lasx_xvpermi_q instrinsic (#82984) (PR #83540)

[Lu Weining via llvm-branch-commits]

SixWeining wrote:

> For the record, based on the principle of "explicit is better than implicit" that generally holds, I'd favor an approach where such compile-time-verifiable out-of-range operands are given compile-time errors, or we should just pass through the value unmodified. Otherwise the intrinsic would have to carry the workaround effectively forever, and we could find ourselves trapped if later micro-architectures actually start to make use of the currently cleared bits.

Makes sense. After discussion with gcc-loongarch team, we both decide to revert this change. Thanks.

https://github.com/llvm/llvm-project/pull/83540