[llvm] [AMDGPU][True16][CodeGen] update GFX11Plus codegen test with true16 flag (PR #135078)

Brox Chen via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 10 06:48:38 PDT 2025


broxigarchen wrote:

> Why do all tests need to check this feature? Why not have a few dedicated test cases just to stress-test this? I'm a bit uneasy seeing the amount of CHECK lines added in some tests (thousands, even 42k for one test). Our test suite can be pretty slow already so if we're going to add this many check lines it has to be worth it, IMO.

Hi Pierre. This is in a transition state. We plan to use +real-true16 mode as default in the end. Thus we added test for both mode now, and eventually we will remove the -real-true16 check line after we switch the default.

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


More information about the llvm-commits mailing list