rjmccall added inline comments.

================
Comment at: clang/test/SemaCXX/PR51855.cpp:71
+// CHECK: call void @_ZN1S4putMEs
+// CHECK: call {{(signext )?i16}} @_ZN1S4getMEv
----------------
rnk wrote:
> Please add test coverage for the interesting operators in the code under 
> edit: OO_Subscript, OO_Arrow, OO_Amp. Please also add equivalent coverage to 
> the ObjC test cases, or give me a link to show that it's already covered.
> 
> Maybe OO_Amp is unnecessary, I'm not sure how it is supposed to work for 
> psuedo objects.
Different pseudo-object extensions can define their own rules for different 
syntactic use patterns, but currently there aren't any that have special 
treatment for either unary or binary `&`; you just take the default "r-value" 
path and then see if you can apply the operator to the result.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111639

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to