[PATCH] D69528: [PowerPC][XCOFF] Add assembly support for zero initalized global values.

[Digger via Phabricator via llvm-commits]

DiggerLin added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp:1861
+  // as tentative defintions, which is only appropriate for SectionKind::Common.
+  if (Kind.isBSS())
+    return DataSection;
----------------
DiggerLin wrote:
> how about the Kind.isBSSExternal() ?
I read the code again, the Kind.isBSS() include the BSSExternal
 bool isBSS() const { return K == BSS || K == BSSLocal || K == BSSExtern; }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69528