[PATCH] D97964: [flang][OpenMP] Add semantic check for occurrence of multiple list items in aligned clause for simd directive

Arnamoy B via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 4 14:54:55 PST 2021


arnamoy10 added a comment.

In D97964#2604649 <https://reviews.llvm.org/D97964#2604649>, @kiranchandramohan wrote:

> Thanks @arnamoy10 for this patch.
>
> Did you try the other approach that I mentioned using FindClause? I am guessing FindClause will only return a single clause and hence cannot be used directly. Would adding another function (if a similar one does not exist) which returns multiple clauses (using equal_range) work? This way we do not need to use the clauseList.

Exactly.  `FindClause` did not work for me for the reason mentioned above.  I had this idea of using `equal_range` as well (and a new function).  Thanks for confirming.  Let me work on that and submit an updated patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97964



More information about the llvm-commits mailing list