Kaydet (Commit) da9fb5d6 authored tarafından Noel Grandin's avatar Noel Grandin

comment out the EvaluateAsInt compat code for now

sberg: Note how that Clang change keeps getting reverted; latest one is
<http://llvm.org/viewvc/llvm-project?view=revision&revision=347656>
"Revert r347417 'Re-Reinstate 347294 with a fix for the failures.'"  But
lets wait a little while to see how things settle there.

Change-Id: I05b34ca1377792bcb4c557defd3e9b5d11ee2b14
Reviewed-on: https://gerrit.libreoffice.org/64168
Tested-by: Jenkins
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
üst 1215b3cf
......@@ -157,7 +157,8 @@ inline bool CPlusPlus17(clang::LangOptions const & opts) {
}
inline bool EvaluateAsInt(clang::Expr const * expr, llvm::APSInt& intRes, const clang::ASTContext& ctx) {
#if CLANG_VERSION >= 80000
// this keeps getting landed and then reverted in clang
#if 0 && CLANG_VERSION >= 80000
clang::Expr::EvalResult res;
bool b = expr->EvaluateAsInt(res, ctx);
if (b && res.Val.isInt())
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment