[PATCH] D142375: [AMDGPU][CodeGen] Combine SDAG and global isel tests for intersect ray intrinsics.
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 8 04:07:21 PST 2023
foad added a comment.
In D142375#4110027 <https://reviews.llvm.org/D142375#4110027>, @kosarev wrote:
> The only differences between the tests discovered are the use of `inreg`
Generally we would not bother with `inreg` except for instruction operands that have to go in SGPRs. But it doesn't really matter. At worst, it will bloat the output with some extra SGPR-to-VGPR copies.
> and flattened arguments instead of vector ones in SDAG tests. I assumed that's what we prefer for the combined tests.
Again it doesn't really matter. Personally I would use vector arguments because it keeps the IR nice and short.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142375/new/
https://reviews.llvm.org/D142375
More information about the llvm-commits
mailing list