ping
================
Comment at: lib/Sema/SemaStmtAsm.cpp:257
@@ -256,3 +256,2 @@
} else if (Info.requiresImmediateConstant() && !Info.allowsRegister()) {
llvm::APSInt Result;
- if (!InputExpr->EvaluateAsInt(Result, Context))
----------------
majnemer wrote:
> I think we could just fix this by inserting:
> if (InputExpr->isValueDependent())
> continue;
Just that does not work because the 'continue' skips needed code later inside
the loop body, leading to assertion failures, on the windows build at least.
However, wrapping whole 10 line checking code in a
(!InputExpr->isValueDependent()) works. I am updating the review to reflect
that.
http://reviews.llvm.org/D10452
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
