================
Comment at: lib/Basic/Targets.cpp:5769
@@ +5768,3 @@
+ .Case("mips32r3", NanLegacy | Nan2008)
+ .Case("mips32r5", NanLegacy | Nan2008)
+ .Case("mips32r6", Nan2008)
----------------
It might be me only, but seeing a function that returns enum but can actually
return non-enumerated value seems wrong (I am not saying it is illegal though).
I see this is already done for ArchDefineTypes within PPCTargetInfo, but it
still looks wrong (to me).
Would not this trigger a runtime error in clang if it was compiled with
"-fsanitize=enum"?
http://reviews.llvm.org/D8170
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
