[PATCH] D29673: [DebugInfo] Append extended dereferencing mechanism to variables' DIExpression for targets that support more than one address space

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: lib/Basic/Targets.cpp:2275
+  /// false otherwise.
+  bool targetSupportsMultipleAddressSpaces() const {
+    return true;
----------------
Is this necessary? Or could it be done only when the address space is non-zero (so the getTargetAddressspace would be called unconditionally, if the result is non-zero add the addr space xderef thing)?


https://reviews.llvm.org/D29673