[PATCH] Make it possible for ints/floats to return different values from getBooleanContents()

hfinkel at anl.gov hfinkel at anl.gov
Wed Jul 9 12:35:37 PDT 2014


Assuming you take my suggestion regarding setBooleanContents (or otherwise make it so that the function-call order does not matter), LGTM. Thanks!

================
Comment at: include/llvm/Target/TargetLowering.h:974
@@ -956,1 +973,3 @@
+  /// See getBooleanContents.
+  void setBooleanFloatContents(BooleanContent Ty) { BooleanFloatContents = Ty; }
 
----------------
I'd really prefer that you do this as I suggested:
void setBooleanContents(BooleanContent Ty) { BooleanContents = Ty; BooleanFloatContents = Ty; }

void setBooleanContents(BooleanContent Ty, BooleanContent FTy) { BooleanContents = Ty; BooleanFloatContents = FTy; }

The reason is that, as you've implemented it, the order in which you call setBooleanContents and setBooleanFloatContents matters, and I think that will prove confusing.

http://reviews.llvm.org/D4389






More information about the llvm-commits mailing list