[PATCH] D114639: Raise the minimum Visual Studio version to VS2019

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 26 09:00:17 PST 2021


RKSimon added inline comments.


================
Comment at: llvm/include/llvm/Support/Compiler.h:106-108
 /// Sadly, this is separate from just rvalue reference support because GCC
 /// and MSVC implemented this later than everything else. This appears to be
 /// corrected in MSVC 2019 but not MSVC 2017.
----------------
jhenderson wrote:
> Does this comment need changing?
An even bigger question is - can we get rid of the LLVM_HAS_RVALUE_REFERENCE_THIS define entirely now? Either as part of this patch or as a followup


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114639/new/

https://reviews.llvm.org/D114639



More information about the cfe-commits mailing list