于 2013/3/8 1:52, Joerg Sonnenberger 写道:
On Fri, Mar 08, 2013 at 12:33:58AM +0800, Jiong Wang wrote:
please review, thanks.
configure.ac: I guess it would be preferable to not enable it by default
for the moment.
I agree.

include/llvm/Support/FEnv.h: What is supposed to handle? Build on/for
Tile? Does the platform provide fenv.h but in a broken way?
yes, the platform provide fenv.h but in a broken way.

TileGX has incomplete support for fenv, we are lack of some macro define.

lib/Target/Mips/MCTargetDesc/MipsMCAsmInfo.cpp: Is this change
intentional?
No. Sorry, my bad, it's not intentional.

---
Regards,
Jiong
Tilera Corporation.


Personally, I would like to see this in the tree sooner than later :)

Joerg
_______________________________________________
LLVM Developers mailing list
[email protected]         http://llvm.cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to