vit9696 added a comment.

Justin, I am currently testing the latest patches, yet so far it looks very 
good. Thanks.
I rechecked GCC, and it seems that it forces 64-bit long double for SPE 
regardless of the target.
Could you please force that in LLVM as well? While imperfect, I currently 
changed the following part of code in clang/lib/Basic/Targets/PPC.cpp, and it 
seems to work for the needs:

  } else if (Feature == "+spe") {
    HasSPE = true;
    LongDoubleWidth = LongDoubleAlign = 64;
    LongDoubleFormat = &llvm::APFloat::IEEEdouble();
  }


Repository:
  rC Clang

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

https://reviews.llvm.org/D49754



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to