mizvekov abandoned this revision. mizvekov added inline comments.
================ Comment at: clang/lib/Sema/SemaExpr.cpp:1135 + QualType RHSElementType = + RHSComplexType ? RHSComplexType->getElementType() : RHSType; + QualType ResultType = ---------------- shafik wrote: > Can you collapse this check with the next line? It just looks weird checking > `RHSComplexType` twice and having `RHSType` on the opposite side of each one. > It took me a while to understand what was going on. > > This comment also applies a few lines down. I have refactored the whole thing, but at D111509 The present patch was intended to be a quick fix to that one, to avoid reverting it, but that is moot now since it ended up reverted and we can't test what is in here without the other one. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133522/new/ https://reviews.llvm.org/D133522 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
