rsmith added a comment.
We've hit a fairly subtle miscompile caused by this patch.
glibc's setjmp.h looks like this (irrelevant parts removed):
struct __jmp_buf_tag { /*...*/ };
extern int __sigsetjmp(struct __jmp_buf_tag __env[1], int);
typedef struct __jmp_buf_tag sigjmp_buf[1];
#defin
njames93 updated this revision to Diff 294895.
njames93 added a comment.
Address reviewer comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82089/new/
https://reviews.llvm.org/D82089
Files:
clang-tools-extra/clang-tidy/modernize/LoopConvert
njames93 updated this revision to Diff 294897.
njames93 marked an inline comment as done.
njames93 added a comment.
IsReverse=
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82089/new/
https://reviews.llvm.org/D82089
Files:
clang-tools-extra/clan
simoll added a comment.
I'd like to hear your feedback on how to proceed with this patch.
Here is what i would do:
For this patch, we switch to ext_vector_type for bool vectors to not clash head
on with GCC should GCC decide to allow vector_size bool with a different
semantics in the future.
F
CarolineConcatto added a comment.
Hi Folks,
@awarzynski and @kiranchandramohan, Diana was working in an out-of-tree build.
This is the patch:
https://reviews.llvm.org/D87085
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87774/new/
https://reviews.
201 - 205 of 205 matches
Mail list logo