arichardson accepted this revision.
arichardson added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/test/SemaCXX/builtin-align-cxx.cpp:40
 
 template <typename T>
 void test_incorrect_alignment_without_instatiation(T value) {
----------------
Could also change this to `template <typename T, long A = 31>` and add another 
` __builtin_align_up(array, A)` but I'm not sure whether that increases test 
coverage.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83074



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to