================

----------------
banach-space wrote:

[nit] Once you have more than one prefix, `CHECK` becomes noise. At least IMHO 
😅  (we all know that these are "check" lines). You could use more descriptive 
prefixes instead, e.g. `CHECK-PORT` -> `PORTABILITY` (it wasn't obvious to me 
that `PORT` stood for "portability"), `CHECK` -> `DEFAULT`.

https://github.com/llvm/llvm-project/pull/90420
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to