[PATCH] Fix .thumb_func directive in ARMAsmParser

Tim Northover t.p.northover at gmail.com
Fri Oct 25 03:06:29 PDT 2013


  Hi Gabor,

  Thanks for doing this. One tiny comment, but other than that it looks fine. Feel free to commit with that change.

  Cheers.

  Tim.


================
Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:77
@@ -76,1 +76,3 @@
 
+  bool nextSymbolIsThumb;
+
----------------
Very minor point: variable names should start with a capital letter in LLVM.


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



More information about the llvm-commits mailing list