[PATCH] D42885: [AMDGPU] intrintrics for byte/short load/store

Neil Henning via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 19 01:55:26 PDT 2018

Previous message: [PATCH] D53400: [clangd] Remove the overflow log.
Next message: [PATCH] D42885: [AMDGPU] intrintrics for byte/short load/store
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

sheredom added a comment.
Herald added a subscriber: jvesely.

Maybe a dumb question - but why can't we just use the tbuffer load/store instead of these? It already upcasts for you (the zext/sext is built in depending on the nfmt I believe).


Repository:
  rL LLVM

https://reviews.llvm.org/D42885

Previous message: [PATCH] D53400: [clangd] Remove the overflow log.
Next message: [PATCH] D42885: [AMDGPU] intrintrics for byte/short load/store
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list