[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    Re: [PATCH] D11305: [Sema] Emit correct warning when copy-elision is not possible.
From:       Richard Smith <richard () metafoo ! co ! uk>
Date:       2015-07-17 20:39:45
Message-ID: 4b0845b892816fb0ed23134620c57eda () localhost ! localdomain
[Download RAW message or body]

rsmith added inline comments.

================
Comment at: test/SemaCXX/warn-pessmizing-move.cpp:158
@@ -165,4 +157,3 @@
 
-// Macro test.  Since the std::move call is outside the macro, it is
-// safe to suggest a fix-it.
+// Macro tests.
 A test8(A a) {
----------------
Do we have more macro tests that test cases where the warning is produced? If not, \
please change the tests to use a local variable.


http://reviews.llvm.org/D11305




_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic