[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