[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