[PATCH] D136545: [Clang] use non-instantiated function declaration for constraints partial ordering

Yuanfang Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 24 22:38:53 PDT 2022


ychen added inline comments.


================
Comment at: clang/test/CXX/over/over.match/over.match.best/p2.cpp:10
   };
-  bool k1 = A<int>() < A<int>(); // not ordered by constraints: prefer non-rewritten form
-  bool k2 = A<float>() < A<float>(); // prefer more-constrained 'operator<=>'
----------------
erichkeane wrote:
> ychen wrote:
> > Remove this case because it compiles by accident. https://eel.is/c++draft/over.match#best.general-2.6 take precedence over https://eel.is/c++draft/over.match#best.general-2.8.
> Could you instead replace it with a better comment + a 'fixme' or something if we need to fix it in the future?  I don't want to lose the test.
Sure. I restored the test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136545



More information about the cfe-commits mailing list