ABataev added inline comments.
================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:15293
+ // TypeDecorCnt for B: 0
+ if (Depth <= TypeDecorCnt) {
+ RelevantExpr = cast<Expr>(DRE);
----------------
cchen wrote:
> ABataev wrote:
> > The check is really bad. If you want to handle something like `*(B+x)`
> > better to treat, say, `B` as the base and `x` as the index and, thus, treat
> > the whole expression as something like `B[x]`
> I don't understand how to do this, is there any sample code that I can learn
> from? Thanks
Not sure we have something like this, need to invent something new if we want
to support lvalues in full. Or discard it as unsupported if we're unable to
support it properly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72811/new/
https://reviews.llvm.org/D72811
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
