myhsu added inline comments.

================
Comment at: clang/lib/Basic/Targets/M68k.cpp:145
   case 'd': // data register
-  case 'f': // floating point register
     info.setAllowsRegister();
----------------
nickdesaulniers wrote:
> did you mean to drop support for `'f'`?
To drop it for now. I believe this constraint was added by mistake since we 
barely has any support for floating point in the backend. I think it's a good 
idea to remove this constraint from the frontend so that it will not be misused 
until floating point implementation got matured.


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

https://reviews.llvm.org/D102585

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

Reply via email to