aaron.ballman added inline comments.

================
Comment at: clang/test/AST/Interp/functions.cpp:158-174
+struct F {
+  constexpr bool ok() const {
+    return okRecurse();
+  }
+  constexpr bool okRecurse() const {
+    return true;
+  }
----------------
Should we have some similar tests involving free functions as well?

Also, how does `BodylessMemberFunction` differ from `F`? They both look to be 
testing the same thing aside from return types.

Should we have a test that involves a defaulted special member function that is 
called explicitly? e.g., https://godbolt.org/z/nzjEcPMKG (Clang is correct 
here, GCC fails to catch the UB).


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

https://reviews.llvm.org/D141591

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

Reply via email to