yaxunl accepted this revision.
yaxunl added a comment.
This revision is now accepted and ready to land.





================
Comment at: lib/AST/ASTContext.cpp:2696
+  ArySize = ArySize.zextOrTrunc(
+      Target->getPreferredPointerWidth(getTargetAddressSpace(EltTy)));
 
----------------
Can we just use getMaxPointerWidth instead? If we cannot determine the pointer 
size, using max pointer size makes sense. Also can you add a ToDo comment about 
this. Ideally we could get a better fix later.


https://reviews.llvm.org/D30845



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

Reply via email to