================
@@ -694,7 +694,8 @@ class ScalarExprEmitter : public
StmtVisitor<ScalarExprEmitter, mlir::Value> {
return {};
}
- if (mlir::isa<cir::SingleType, cir::DoubleType>(value.getType())) {
+ if (mlir::isa<cir::SingleType, cir::DoubleType, cir::LongDoubleType>(
+ value.getType())) {
----------------
erichkeane wrote:
I really don't think so. I'm not sure why this is formed this way, it was
added in https://github.com/llvm/llvm-project/pull/131369
So I was hoping you'd have some insight :)
At the moment, I suspect that this adds value for the sole fact that it makes
us look at what 'half' will do in the lowering (and I very much suspect said
'half' enable patch will be very similar to this one if the lowering doesn't
require anything further).
https://github.com/llvm/llvm-project/pull/190812
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
