[PATCH] D141591: [clang][Interp] Properly identify not-yet-defined functions

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 31 05:26:18 PST 2023


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



More information about the cfe-commits mailing list