[PATCH] D134902: [clang] Implement -fstrict-flex-arrays=3

serge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 21 23:42:25 PDT 2022


serge-sans-paille added inline comments.


================
Comment at: clang/include/clang/Basic/LangOptions.h:60
 ///  static_asserts in Triple.cpp and in clang/Basic/HLSLRuntime.h.
 enum class ShaderStage {
   Pixel = 0,
----------------
Looks unrelated to this patch :-)


================
Comment at: clang/include/clang/Basic/LangOptions.h:231
     ///   - Reverse the implementation for DR692, DR1395 and DR1432.
     ///   - pack non-POD members of packed structs.
     Ver15,
----------------
Looks unrelated to this patch.


================
Comment at: clang/lib/AST/ExprConstant.cpp:14085
     return true;
   }
   }
----------------
fascinatingly off topic. But gives a good hint on what you're working on ;-) 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134902



More information about the cfe-commits mailing list