t.p.northover accepted this revision.
t.p.northover marked an inline comment as done.
t.p.northover added a comment.

Thanks Reid, committed as r367755.



================
Comment at: llvm/utils/add_argument_names.py:5
+def fix_string(s):
+    TYPE = 
re.compile('\s*(i[0-9]+|float|double|x86_fp80|fp128|ppc_fp128|\[\[.*?\]\]|\[2 x 
\[\[[A-Z_0-9]+\]\]\]|<.*?>|{.*?}|\[[0-9]+ x 
.*?\]|%["a-z:A-Z0-9._]+({{.*?}})?|%{{.*?}}|{{.*?}}|\[\[.*?\]\])(\s*(\*|addrspace\(.*?\)|dereferenceable\(.*?\)|byval\(.*?\)|sret|zeroext|inreg|returned|signext|nocapture|align
 \d+|swiftself|swifterror|readonly|noalias|inalloca|nocapture))*\s*')
+
----------------
rnk wrote:
> Nice.
Incoming prize for the most egregious violation of the 70 column rule.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65582/new/

https://reviews.llvm.org/D65582



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

Reply via email to