[PATCH] D20840: [codeview] Translate basic DITypes to CV type records

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Tue May 31 16:06:08 PDT 2016


zturner added inline comments.

================
Comment at: lib/CodeGen/AsmPrinter/CodeViewDebug.cpp:857-859
@@ +856,5 @@
+  }
+  // FIXME: MSVC folds qualifiers into PointerOptions in the context of a method
+  // 'this' pointer, but not normal contexts. Figure out what we're supposed to
+  // do.
+  PointerOptions PO = PointerOptions::None;
----------------
Is this true?  Surely there has to be a `PointerOptions` on a non member pointer.


http://reviews.llvm.org/D20840





More information about the llvm-commits mailing list