[PATCH] D13834: Produce a better diagnostic for global register variables
Eric Christopher via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 9 15:08:17 PST 2015
echristo added a comment.
My preference for this sort of thing would be an enum of failure reasons, but I guess this is ok for now. One inline request.
-eric
================
Comment at: lib/Basic/Targets.cpp:3978-3981
@@ +3977,6 @@
+ bool &HasSizeMismatch) const override {
+ if (RegName.equals("rsp") || RegName.equals("rbp")) {
+ HasSizeMismatch = RegSize != 64;
+ return true;
+ }
+
----------------
This could use a comment, what are you testing here?
http://reviews.llvm.org/D13834
More information about the cfe-commits
mailing list