[PATCH] D44401: [AMDGPU] Always use IDX for load/store format intrinsics.
    Marek Olšák via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Mar 16 13:31:14 PDT 2018
    
    
  
mareko added a comment.
In https://reviews.llvm.org/D44401#1040175, @arsenm wrote:
> I think the unit change is a hint we should use different intrinsics for this
The current intrinsic is unusable on GFX9 though. Why should we keep it as-is if it's unusable? This patch is reasonable because it picks the most expected behavior for GFX9 while it disallows the least expected and problematic behavior.
Repository:
  rL LLVM
https://reviews.llvm.org/D44401
    
    
More information about the llvm-commits
mailing list