[PATCH] MS ABI: Implement #pragma vtordisp() and clang-cl /vdN

David Majnemer david.majnemer at gmail.com
Wed Feb 12 15:08:11 PST 2014


  LGTM


================
Comment at: lib/Lex/Preprocessor.cpp:832
@@ +831,3 @@
+    return false;
+  llvm::APInt APVal(32, 0);
+  if (Literal.GetIntegerValue(APVal))
----------------
`Value` is 64-bit, shouldn't this be `ApVal(32, 0)`  ?


http://llvm-reviews.chandlerc.com/D2746

BRANCH
  pragma-vtordisp

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list