[llvm] [AMDGPU] Update instrumentAddress method to support aligned size and unusual size accesses. (PR #104804)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 19 08:47:06 PDT 2024
================
@@ -179,6 +181,46 @@ void instrumentAddress(Module &M, IRBuilder<> &IRB, Instruction *OrigIns,
return;
}
+void instrumentAddress(Module &M, IRBuilder<> &IRB, Instruction *OrigIns,
+ Instruction *InsertBefore, Value *Addr,
+ MaybeAlign Alignment, TypeSize TypeStoreSize,
+ bool IsWrite, Value *SizeArgument, bool UseCalls,
+ bool Recover, int AsanScale, int AsanOffset) {
+ // Instrument a 1-, 2-, 4-, 8-, or 16- byte access with one check
+ // if the data is properly aligned.
----------------
arsenm wrote:
We have 12 byte accesses too. Plus I don't see why you need to special case any of this and not just emit the access as-is
https://github.com/llvm/llvm-project/pull/104804
More information about the llvm-commits
mailing list