ABataev added inline comments.
================ Comment at: clang/lib/Sema/SemaOpenMP.cpp:15684-15685 + bool VisitUnaryOperator(UnaryOperator *UO) { + if (SemaRef.getLangOpts().OpenMP < 50 || UO->getOpcode() != UO_Deref || + !UO->isLValue()) { + emitErrorMsg(); ---------------- `UO_Deref` is always r-value. So, `!UO->isLValue()` will be always triggered for `UO_Deref`. You can just exclude it from the condition. ================ Comment at: clang/test/OpenMP/target_update_codegen.cpp:375 + // CK6-DAG: [[L_VAL]] = load i32, i32* [[L_ADDR:%.+]] + // CK6-DAG: store i32 [[L:%.+]], i32* [[L_ADDR]] + #pragma omp target update to(*(B+l)) ---------------- Seems to me, `L` is not used in the test, so you can just use `{{%.+}}` here Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75077/new/ https://reviews.llvm.org/D75077 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits