saugustine added a comment.
Landed as r309973
Repository:
rL LLVM
https://reviews.llvm.org/D36249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.
I'm ok with this.
Repository:
rL LLVM
https://reviews.llvm.org/D36249
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists
weimingz added a comment.
In https://reviews.llvm.org/D36249#830645, @saugustine wrote:
> In https://reviews.llvm.org/D36249#830121, @weimingz wrote:
>
> > I tried to address it via checking pre-defined macros:
> > https://reviews.llvm.org/D31573
> >
> > As long as the macros are defined correct
saugustine added a comment.
In https://reviews.llvm.org/D36249#830121, @weimingz wrote:
> I tried to address it via checking pre-defined macros:
> https://reviews.llvm.org/D31573
>
> As long as the macros are defined correctly by clang, we don't need to worry
> about the specific target machine
weimingz added a comment.
I tried to address it via checking pre-defined macros:
https://reviews.llvm.org/D31573
As long as the macros are defined correctly by clang, we don't need to worry
about the specific target machine. How do you think about it?
Repository:
rL LLVM
https://reviews.llv
saugustine created this revision.
These tests assume intel-80 bit floats, as noted in their comments. Rather than
having each target discover the incompatibility and disable them ad-hoc, mark
them as requiring x86.
Repository:
rL LLVM
https://reviews.llvm.org/D36249
Files:
compiler-rt/te
