This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG03dff0d4acaf: [clang-tidy] Disambiguate calls to log (authored by ro).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158959/new/ https://reviews.llvm.org/D158959 Files: clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp Index: clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp =================================================================== --- clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp +++ clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp @@ -214,10 +214,12 @@ Iterations = ceil(float(InitValue - EndValue) / ConstantValue); break; case (BO_MulAssign): - Iterations = 1 + (log(EndValue) - log(InitValue)) / log(ConstantValue); + Iterations = 1 + (log((double)EndValue) - log((double)InitValue)) / + log((double)ConstantValue); break; case (BO_DivAssign): - Iterations = 1 + (log(InitValue) - log(EndValue)) / log(ConstantValue); + Iterations = 1 + (log((double)InitValue) - log((double)EndValue)) / + log((double)ConstantValue); break; default: // All other operators are not handled; assume large bounds.
Index: clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp =================================================================== --- clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp +++ clang-tools-extra/clang-tidy/altera/UnrollLoopsCheck.cpp @@ -214,10 +214,12 @@ Iterations = ceil(float(InitValue - EndValue) / ConstantValue); break; case (BO_MulAssign): - Iterations = 1 + (log(EndValue) - log(InitValue)) / log(ConstantValue); + Iterations = 1 + (log((double)EndValue) - log((double)InitValue)) / + log((double)ConstantValue); break; case (BO_DivAssign): - Iterations = 1 + (log(InitValue) - log(EndValue)) / log(ConstantValue); + Iterations = 1 + (log((double)InitValue) - log((double)EndValue)) / + log((double)ConstantValue); break; default: // All other operators are not handled; assume large bounds.
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits