erichkeane wrote: > > > errors, but I think to fold it directly we'd need a dedicated builtin. > > > But I'm not a master of the clang AST. > > > > > > We shouldn't need a builtin, we should just be able to handle the AST nodes > > directly in ExprConstant.cpp. > > Ah, right. That doesn't work, but it seems like the normal handling doesn't > work there either so I'll punt fixing this for another PR > https://godbolt.org/z/rTx5n7fao.
I expected as much :) This suggestion: > Another thing to look at is handling these operations in ExprConstant.cpp > so that we handle them in constexpr too! was intended as a future direction (or a nerd-snipe 'while you are here?') sorry if that wasn't clear enough https://github.com/llvm/llvm-project/pull/154145 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
