Merge pull request #1092 from Shishpan/ternaryFix

Added unsafe case of division with "all on" mask, fix for #1083.
2015-08-21 19:14:31 +03:00
parent 5d509b69c8 bcdef9ea71
commit e164804f43
1 changed files with 7 additions and 0 deletions
--- a/ast.cpp
+++ b/ast.cpp
@@ -420,6 +420,13 @@ lCheckAllOffSafety(ASTNode *node, void *data) {
        return false;
    }

+    if (llvm::dyn_cast<BinaryExpr>(node) != NULL) {
+        BinaryExpr* binaryExpr = llvm::dyn_cast<BinaryExpr>(node);
+        if (binaryExpr->op == BinaryExpr::Mod || binaryExpr->op == BinaryExpr::Div) {
+            *okPtr = false;
+            return false;
+        }
+    }
    IndexExpr *ie;
    if ((ie = llvm::dyn_cast<IndexExpr>(node)) != NULL && ie->baseExpr != NULL) {
        const Type *type = ie->baseExpr->GetType();