[PATCH] D125164: [X86] Fix some signedness errors in x86 headers

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 7 08:24:17 PDT 2022


RKSimon added inline comments.


================
Comment at: clang/lib/Headers/cetintrin.h:37
 static __inline__ void __DEFAULT_FN_ATTRS _inc_ssp(unsigned int __a) {
-  __builtin_ia32_incsspd((int)__a);
+  __builtin_ia32_incsspd((unsigned int)__a);
 }
----------------
pengfei wrote:
> Unnecessary cast?
yeah, we can just remove the (int)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125164/new/

https://reviews.llvm.org/D125164



More information about the cfe-commits mailing list