[PATCH] Fix for PR14269: Clang crashes when a bit field is used as inline assembler input / output with memory constraint

Eric Christopher echristo at gmail.com
Thu Jun 4 17:21:10 PDT 2015


One inline comment and then LGTM.

Thanks!

-eric


================
Comment at: lib/Sema/SemaStmtAsm.cpp:241
@@ -232,1 +240,3 @@
 
+    // Bit field can't be referenced with a pointer.
+    if (Info.allowsMemory() && InputExpr->refersToBitField())
----------------
"Bit field" -> "Bitfield" please.

http://reviews.llvm.org/D10086

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list