aaron.ballman added a comment.

"Unsupported" is a bit of a loaded term -- it could mean "this operation is not 
supported, YET" or it could mean "this operation is not and will not be 
supported, EVER". Perhaps something more like "InvalidInConstantExpr" would be 
more descriptive?



================
Comment at: clang/test/AST/Interp/unsupported.cpp:3-5
+
+
+
----------------
Can get rid of some whitespace.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150364

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

Reply via email to