kcc accepted this revision. kcc added a comment. This revision is now accepted and ready to land.
LGTM with a nit ================ Comment at: test/CodeGen/cfi-icall.c:1 @@ +1,2 @@ +// RUN: %clang_cc1 -triple x86_64-unknown-linux -fsanitize=cfi-icall -fsanitize-trap=cfi-icall -emit-llvm -o - %s | FileCheck --check-prefix=ITANIUM %s +// RUN: %clang_cc1 -triple x86_64-pc-windows-msvc -fsanitize=cfi-icall -fsanitize-trap=cfi-icall -emit-llvm -o - %s | FileCheck --check-prefix=MS %s ---------------- Add a comment explaining what we are testing here. same below. http://reviews.llvm.org/D11857 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits