[PATCH] D29671: [DebugInfo] Add address space when creating DIDerivedTypes

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: include/clang/Basic/TargetInfo.h:1043
+  virtual Optional<unsigned> getDWARFAddressSpace(unsigned AddressSpace) const {
+    return None;
+  }
----------------
Is this a reasonable default implementation? Should there be a default or should any implementation (mabe only those with non-zero address spaces?) be expected to provide a mapping? (not sure how this looks for other things in TargetInfo)


https://reviews.llvm.org/D29671