aaron.ballman added a comment. In D133583#3780460 <https://reviews.llvm.org/D133583#3780460>, @yronglin wrote:
> Hi, follow D133202 <https://reviews.llvm.org/D133202> , should I both fix > alignment in this patch or in another separate patch? (this seems have > different behavior with gcc https://godbolt.org/z/7dvM8zhnh ) I think that's a separate patch -- this one is fixing a mistake with the type-checking related crash fix and the other is about the behavior of the expression itself when it's valid. The current patch looks reasonable to me, but I'd love a second opinion. ================ Comment at: clang/test/Sema/builtin-redecl.cpp:5-6 +#include <stddef.h> + // Redeclaring library builtins is OK. ---------------- We can be tricky instead of including a header file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133583/new/ https://reviews.llvm.org/D133583 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits