jfb added a comment.

Can you change the title to be more descriptive?

Small test fixes, otherwise this LGTM, will check box after update.



================
Comment at: clang/test/CodeGenCXX/auto-var-init.cpp:123
+// PATTERN-NOT-O1: @__const.test_int1_custom.custom
+// ZERO-NOT-O1: @__const.test_int1_custom.custom
+
----------------

`-NOT` is in the wrong place above.


================
Comment at: clang/test/CodeGenCXX/auto-var-init.cpp:130
+// PATTERN-NOT-O1: @__const.test_bool4_custom.custom
+// ZERO-NOT-O1: @__const.test_bool4_custom.custom
+
----------------
`-NOT` is in the wrong place above.


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

https://reviews.llvm.org/D57898



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

Reply via email to