[PATCH] New SystemZ backend: clang changes

Eric Christopher echristo at gmail.com
Wed Apr 24 05:41:16 PDT 2013


  Might be nice to see some tests for the inline assembly (front and backend if there weren't any there). The Target stuff LGTM otherwise. Someone else will need to get the rest of the changes.


================
Comment at: include/clang/AST/ASTContext.h:1583
@@ +1582,3 @@
+  /// global variable with type \p T.
+  unsigned getGlobalAlign(QualType T) const;
+
----------------
Sadly I think most of the rest of llvm is using bits for alignment sizes, so you'll probably want to use bits here?


http://llvm-reviews.chandlerc.com/D718



More information about the cfe-commits mailing list