Address comments

intel · Dec 26, 2024 · 61bdcc8 · 61bdcc8
1 parent fe1df2b
commit 61bdcc8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/libdevice/sanitizer/msan_rtl.cpp b/libdevice/sanitizer/msan_rtl.cpp
@@ -205,9 +205,9 @@ DEVICE_EXTERN_C_NOINLINE uptr __msan_get_shadow(uptr addr, uint32_t as) {
     }                                                                          \
   }
 
-MSAN_MEMSET(0);
-MSAN_MEMSET(1);
-MSAN_MEMSET(3);
+MSAN_MEMSET(0)
+MSAN_MEMSET(1)
+MSAN_MEMSET(3)
 MSAN_MEMSET(4)
 
 #endif // __SPIR__ || __SPIRV__
diff --git a/llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp b/llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp
@@ -1081,7 +1081,7 @@ void MemorySanitizer::initializeCallbacks(Module &M,
       const std::string Suffix = "_p" + itostr(FirstArgAS);
       PointerType *FirstArgPtrTy = IRB.getPtrTy(FirstArgAS);
       MemsetOffloadFn[FirstArgAS] = M.getOrInsertFunction(
-          "__msan_memset" + Suffix, TLI.getAttrList(C, {1}, true),
+          "__msan_memset" + Suffix, TLI.getAttrList(C, {1}, /*Signed=*/true),
           FirstArgPtrTy, FirstArgPtrTy, IRB.getInt32Ty(), IntptrTy);
     }
   }