[PATCH] D24986: [MS] Implement __iso_volatile loads/stores as builtins

David Majnemer via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 30 10:51:59 PDT 2016


majnemer accepted this revision.
majnemer added a comment.
This revision is now accepted and ready to land.

LGTM



> ms-volatile-arm.c:2
> +// REQUIRES: arm-registered-target
> +// RUN: %clang_cc1 -triple thumbv7-win32 -emit-llvm -fms-extensions -fms-volatile -o - < %s | FileCheck %s
> +

You don't need -fms-volatile.

https://reviews.llvm.org/D24986





More information about the cfe-commits mailing list