- User Since
- Nov 6 2015, 10:28 AM (219 w, 20 h)
Thu, Jan 2
Committed as 108daf76118e5b97696f58386d0b48d4b858ffad
Mon, Dec 30
Fri, Dec 27
Submitted as 1d891a32cf433e205b5d30697beb265dc0e1f248
Dec 18 2019
Landed as 5789e83dedb97588ad75cca36d01ba6c5142d6d3
Dec 6 2019
Committed aa3c877fb58db238a86e734511bdeed9bc40086c to fix the unused variable warning at AArch64InstructionSelector.cpp:1013
This change fails to build when assertions are turned off, due to the variable Ty at Arch64InstructionSelector.cpp:1013 not being used anywhere but an assertion. I'm testing a patch to fix.
Dec 5 2019
When I try to run tablegen on this file for clang/docs (as described in clang/docs/InternalsManual.rst), I get an error, introduced by this change:
Dec 4 2019
I have reverted this due to the asan failures. f65267e is the commit.
I have provided a very reduced test case on the original patch.
Here is the very reduced test case:
This change is causing clang to crash on some internal tests we have.
Dec 2 2019
I'm not a cmake expert and have very little sense of the mechanics here, but this change is necessary and at least looks correct to me, so accepting.
Nov 22 2019
Nov 19 2019
Nov 18 2019
Nov 12 2019
Move prototypes for the frame info functions as requested.
...and finally, after some off line feedback, someone pointed out to me that we really do want using the frame registry to be configurable. Not the least of which because llvm-libunwind does nothing inside __register_frame_info.
- Make using the eh_frame registry a configure-time option.
Nov 11 2019
- Fix space.
In the original code, register_frame_info was always called. But in the original version of this patch, I awkwardly tied it to CRT_USE_INITFINI_ARRAY, which would make it uncalled in certain circumstances. Also, we agreed that tying the two things together didn't make a lot of sense.
- Make USE_FRAME_REGISTRY completely independent of CRT_HAS_INITFINI_ARRAY,
- Address upstream comments. Move backward compatibility handling to CMakeLists.
Nov 8 2019
Nov 7 2019
Updated for all comments, and committing shortly.
- Add file header and explanatory comment.
Nov 6 2019
Nov 5 2019
Nov 1 2019
Oct 31 2019
Committed in rG118ceea5c364, with the git transition obscuring the auto-reporting.
Oct 25 2019
Switch to new repo.
Oct 24 2019
- Avoid a string copy.
- Add test. Address other comments.
- remove llvm namespace.
All comments addressed. Please take another look when you get a chance.
- Address additional comments.
Oct 22 2019
Thanks. Please take another look.
- Switch crt file matching to regular expressions.
Oct 17 2019
Committed. as r375174.
This will be fixed by https://reviews.llvm.org/D69143
This breaks the build due to lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp:1355:8: error: unused variable 'old_size' [-Werror,-Wunused-variable]
Committed as r375148.
https://reviews.llvm.org/D69123 fixes this break.
This revision breaks builds with assertions disabled because IsPPC64 (introduced on line 539 of PPCAsmPrinter.cpp is unused when assertions are turned off.
Oct 4 2019
Oct 3 2019
Oct 2 2019
Yvan Roux had been working on this (python 3 compatibility), and even showed me a brief patch off line. I don't know what happened there. I will cc him now.
Oct 1 2019
Sep 20 2019
We've been blocked on this for a day now, so going ahead and checking it in. Will wait and watch for problems.
Move test case out of GlobalISel.
Trying again in https://reviews.llvm.org/D67850
Sep 19 2019
I fixed our problem with r372366.