[PATCH] D102400: [flang] [openmp] Add Fortran specific semantic check 4 for OpenMP Allocate directive.

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 19 01:02:25 PDT 2021


kiranchandramohan added a comment.

Thanks @I_Perry for the changes. Do you know why the allocate08 test fails?



================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:659
+  const auto &allocatorValue{GetIntValue(x.v)};
+  if (*allocatorValue > 8)
+    isPredefinedAllocator = false;
----------------
We should not hardcode this number 8.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102400



More information about the llvm-commits mailing list