compnerd added a subscriber: compnerd.

================
Comment at: lib/Headers/Intrin.h:961
@@ +960,3 @@
+static __inline__ void __DEFAULT_FN_ATTRS _lgdt(void *__ptr) {
+  __builtin_ia32_lgdt(__ptr);
+}
----------------
Why does this need a builtin?  Is an inline assembly block using lgdt 
insufficient for some reason?

================
Comment at: lib/Headers/Intrin.h:965
@@ +964,3 @@
+static __inline__ void __DEFAULT_FN_ATTRS _sgdt(void *__ptr) {
+  __builtin_ia32_sgdt(__ptr);
+}
----------------
Similar to lgdt.


http://reviews.llvm.org/D12278



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to