lkail added inline comments.
================
Comment at: clang/lib/Basic/Targets/PPC.h:440
if (Triple.isOSAIX() || Triple.isOSLinux())
DataLayout += "-S128-v256:256:256-v512:512:512";
resetDataLayout(DataLayout);
----------------
adalava wrote:
> I don't understand what DataLayout does mean, but I'm wondering why FreeBSD
> doesn't append this as well. Is it expected to be AIX() and Linux only?
As maskray has pointed out, usually using `isOSBinFormatELF` rather than
`isOSLinux` can cover freebsd as well. This snippet of code might be beyond
this patch, we might post another one to address this issue.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122377/new/
https://reviews.llvm.org/D122377
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
