[llvm] [AMDGPU] add tests for loop definition of bitconvert (PR #133052)

via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 28 01:31:46 PDT 2025


Shoreshen wrote:

> > Also what are these lit tests for ? why do we need to test all widths?
> 
> Every operation with every type should be tested.
> 
> For this case, we already should have test coverage so I'm not sure I understand what is new here. We should perhaps split / replace the existing test with a more consistent pattern, but this patch is purely additive?

Hi @arsenm , I removed the old tests and split them into different files named by bit width. 

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


More information about the llvm-commits mailing list