inclyc added inline comments.
================ Comment at: clang/lib/Sema/SemaExprCXX.cpp:5445 + } +} + ---------------- inclyc wrote: > tbaeder wrote: > > This function copies the two conditions for the if-statements from > > `evaluateTypeTrait()`. Is just doing it in `evaluateTypeTrait()` too late? > > Duplicating that seems bad. > `evaluateTypeTrait()` should return evaluate result, i.e. bool value `true` / > `false`, if we do this in `evaluateTypeTrait()` , how can we express that we > encountered some error? I think it is reasonable to split these conditions > here. Do you think we should add a condition checks to the evaluate function `evaluateTypeTrait()` and return `false` if it is not satisfied? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131423/new/ https://reviews.llvm.org/D131423 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits