[clang] [clang] The ms-extension __noop should return zero in a constexpr context. (PR #106849)

[Timm Baeder via cfe-commits]

================
@@ -12720,8 +12720,8 @@ bool IntExprEvaluator::VisitBuiltinCallExpr(const 
CallExpr *E,
   }
 
   case Builtin::BI__noop:
-  // __noop always evaluates successfully
-    return true;
+    // __noop always evaluates successfully
+    return ZeroInitialization(E);
----------------
tbaederr wrote:

```suggestion
    // __noop always evaluates successfully and returns 0.
    return Success(0, E);
```

https://github.com/llvm/llvm-project/pull/106849
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits