lildmh marked an inline comment as done. lildmh added inline comments.
================ Comment at: lib/CodeGen/CGOpenMPRuntime.h:2110 + /// Emit code for the user defined mapper construct. + void emitUserDefinedMapper(const OMPDeclareMapperDecl *D, + CodeGenFunction *CGF = nullptr) override; ---------------- ABataev wrote: > ABataev wrote: > > lildmh wrote: > > > ABataev wrote: > > > > I think you can drop this function here if the original function is not > > > > virtual > > > The function for simd only mode includes a `llvm_unreachable`, so I think > > > it's still needed as a virtual function above > > It is better to reduce number of virtual functions, if possible. > What about virtual function? Since `emitUserDefinedMapper` here overrides the same function in `CGOpenMPRuntime`, I think `emitUserDefinedMapper` of `CGOpenMPRuntime` needs to be defined as `virtual`? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59474/new/ https://reviews.llvm.org/D59474 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
