- User Since
- Nov 19 2012, 2:50 AM (226 w, 6 d)
Fri, Mar 24
Thu, Mar 23
It's part of already-accepted patch ((https://reviews.llvm.org/D30802)
Will re-land it.
Wed, Mar 22
need to include builtin-config-ix to get the right ARM32 variable.
This patch *should* address the issue of buildbot of powerpc (http://lab.llvm.org:8011/builders/sanitizer-ppc64le-linux/builds/1703).
I was using
It should be
Tue, Mar 21
Mon, Mar 20
I split the original patch into 2 parts: This one only contains the test case updates: adding "RUN" or "UNSUPPORT". A couple of test cases are XFAILed for armhf/aarch64 target (bugs were opened [1,2]). This part is also safe to commit; The second part will includes the CMake, lit.cfg files for easy reviewing.
Tue, Mar 14
(sorry, please ignore the update)
Mon, Mar 13
Sun, Mar 12
Sat, Mar 11
Most of the changes are guarded by __ARM_ARCH_ISA_THUMB == 1 and I reviewed again of the "else" part and they look OK except for one suspect:
Fri, Mar 10
Thanks for trying it on x86, Michal.
I XFAILed cpu_model_test.c and gcc_personality_test.c for now and I leave "FIXME" around the %RUN line. We can fix them later on.
Address Jonathan's comments.
Just rebased and find that https://reviews.llvm.org/D30136 is adding TestCase support.
I'm not sure what's the TestCase is used for. Looks only for Darwin and the directory is empty.
Thu, Mar 9
I tested it on ARM via QEMU with the CMAKE parames below:
will use lit-test
Tue, Mar 7
Fri, Feb 24
Sure, I will get more numbers for other benchmarks.
Feb 23 2017
Feb 22 2017
Feb 15 2017
Feb 14 2017
Feb 12 2017
Feb 11 2017
Feb 5 2017
Feb 3 2017
My previous comment of "136 bytes of stack usage" is inaccurate. I just find the default build doesn't have any -O flag. Is it an issue without a default -O flag?
Feb 2 2017
Jan 30 2017
Jan 27 2017
Jan 26 2017
I run the tests under compiler-rt/test/builtins/Unit/arm
using lsls to indirectly set PSR flags
Jan 23 2017
As Eli sugguested, it's better to check it in Sema. In order to access RelocationModel in Sema, we moved it from CodeGenOpts to LangOpts
Jan 21 2017
Jan 20 2017
I included the fix in https://reviews.llvm.org/D28971
Jan 19 2017
Jan 18 2017
Jan 11 2017
Jan 10 2017
Thanks for the comments!
I though of implementation in Sema and I actually experimented a little bit. Then it seemed that the CodeGenOpts is only available in the CGM. Any ideas?
Ugh, I missed them in during clean up in diff 2. Very sorry!
Jan 8 2017
Jan 7 2017
I can fix the comments
Jan 5 2017
Manually ran the unit test and passed
Dec 28 2016
Missed a format issue in previous upload. Sorry~
By tracking the original implementation, I think we can avoid the msr/mrs. Also address some formatting issues.
Dec 22 2016
Dec 21 2016
The patch looks good to me.
Thanks for the formatting. Regarding the refactoring, Regarding factoring out common code, I think it's OK to do it in a follow-up patch. For readability, it should be OK. For example, fp_add_impl.inc is used by float, double and long double.