[PATCH] D32982: CodeGenModule: Always output wchar_size; check LLVM assumptions.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 17 15:07:52 PDT 2017


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

I guess this is better than DataLayout. Also, the user experience of mixing two TUs with different -fshort-wchar settings during LTO is better.



================
Comment at: lib/CodeGen/CodeGenModule.cpp:473
+         llvm::TargetLibraryInfoImpl::getTargetWCharSize(Target.getTriple())
+         == Target.getWCharWidth()/8 && "LLVM wchar_t size out of sync");
+
----------------
Is this what clang-format does?


Repository:
  rL LLVM

https://reviews.llvm.org/D32982





More information about the llvm-commits mailing list