[PATCH] D71969: [OpenMP] diagnose zero-length array section in the depend clause

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 30 07:26:03 PST 2019


ABataev added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15031-15033
+            Length->EvaluateAsInt(Result, Context)) {
+          llvm::APSInt ConstLength = Result.Val.getInt();
+          if (ConstLength.getSExtValue() == 0) {
----------------
Just `.. Length->EvaluateAsInt(Result, Context) && Result.Val.getInt().isNullValue()) {`


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

https://reviews.llvm.org/D71969





More information about the cfe-commits mailing list