Re: [PATCH] D23953: OpenCL: Defining __ENDIAN_LITTLE__ and fix target endianness

2016-09-07 Thread Matt Arsenault via cfe-commits
arsenm closed this revision. arsenm added a comment. r280787 https://reviews.llvm.org/D23953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D23953: OpenCL: Defining __ENDIAN_LITTLE__ and fix target endianness

2016-09-06 Thread Anastasia Stulova via cfe-commits
Anastasia added a subscriber: Anastasia. Anastasia accepted this revision. Anastasia added a reviewer: Anastasia. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! https://reviews.llvm.org/D23953 ___ cfe-commits maili

Re: [PATCH] D23953: OpenCL: Defining __ENDIAN_LITTLE__ and fix target endianness

2016-09-06 Thread Matt Arsenault via cfe-commits
arsenm added a comment. ping https://reviews.llvm.org/D23953 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D23953: OpenCL: Defining __ENDIAN_LITTLE__ and fix target endianness

2016-08-26 Thread Matt Arsenault via cfe-commits
arsenm updated this revision to Diff 69465. arsenm added a comment. Fix typo https://reviews.llvm.org/D23953 Files: include/clang/Basic/TargetInfo.h lib/Basic/TargetInfo.cpp lib/Basic/Targets.cpp lib/Frontend/InitPreprocessor.cpp test/Preprocessor/init.c Index: test/Preprocessor/init

[PATCH] D23953: OpenCL: Defining __ENDIAN_LITTLE__ and fix target endianness

2016-08-26 Thread Matt Arsenault via cfe-commits
arsenm created this revision. arsenm added a subscriber: cfe-commits. OpenCL requires __ENDIAN_LITTLE__ be set for little endian targets. The default for targets was also apparently big endian, so AMDGPU was incorrectly reported as big endian. Set this from the triple so targets don't have another