Mostly LLVM work.
- User Since
- Nov 2 2018, 7:48 AM (127 w, 5 d)
Wed, Apr 7
Could you please clean up the test a bit? It contains references to attribute groups that don't actually exist (#0, #1). One attribute you do want to add is nounwind, to avoid the clutter caused by the CFI directives. In manually written tests we also generally don't include some of those dso_local, local_unnamed_addr, etc. In general, it would be nice to make it look less like Clang's output and more like something that can be easily read and reasoned about.
Optimize address mappings.
Tue, Apr 6
Mon, Apr 5
Once update_mir_test_checks.py is tweaked we can start using it for this test, but let's merge it as-is for now. Thanks!
Sun, Apr 4
Just a heads-up about a possible pitfall of this patch, though I'm not sure it actually affects SystemZ.
Fixing alignment before truly abandoning, in case it's useful for D97514.
BTW, it shouldn't affect correctness but the original CreateStackTemporary would use (as part of that method's implementation) getPrefTypeAlign instead of getEVTAlign. In principle that would allow better alignment choices, although at the moment it probably would never make any difference.
Thu, Apr 1
Wed, Mar 31
Tue, Mar 30
Address review feedback (thanks @MaskRay!).
Tested with the sanitizer-x86_64-linux-android and sanitizer-windows buildbots, so this time there shouldn't be buildbot failures when committing.
- Add VarArg handling.
- Adjust tests.
Tue, Mar 23
@sequencer What's the plan for this patch?
There are certainly fewer changes to existing tests, compared with
Mon, Mar 22
- Precommitted the original test, to better show the difference caused by of the patch.
- Updated the existing tests. They seem to more broadly show the issue I noted about the excessive alignment/overly conservative stack size.
Sat, Mar 20
Superseded by D77384 and D97490.
I can confirm that the mips* XFAIL now passes (since D77384 landed on Feb 26). If wasm contributors are still interested in reviewing this patch I can update it to include only the immediate.ll test update.
Thu, Mar 18
I just looked at this again and I don't have the full context in my mind right now but won't the test just exercise the BareMetal toolchain and not your changes?
Wed, Mar 17
Address review feedback.
Before relanding, I need to check/fix the SizeClassAllocator64PopulateFreeListOOM test for Android x86, as the respective buildbot failed in the first landing attempt.
Fix silly mistake in the last update.
Tue, Mar 16
This still doesn't report that the multilib configuration came from GCC when it succeeds, does it? I suppose that's not a deal-breaker, but it would be nice to have. Would it be difficult to implement?
Mar 15 2021
This broke debug info for RISC-V. Test case with broken symbolization:
This makes sense to me but I'm not quite sure about the implications, especially when we consider compatibility. I think we need more eyes on this patch.
(Perhaps it would have been better to have split it into the NFC patch and the rest?)
LGTM. Nice clean-up!
Mar 14 2021
This patch seems to be in pretty good shape now.
Why not just error out if passed "generic"? The error message could indicate that "generic-rv<XLEN>" should be used instead.
In other words, it's not clear to me what we're getting with the remap, particularly since it still prints a warning...?
I just don't understand what you mean with "1-byte NOPs" in the patchable prefix case. Regular NOPs are emitted. Please clarify the comment/patch as needed.
Mar 8 2021
Address review feedback.
Mar 7 2021
Mar 4 2021
Mar 3 2021
Mar 1 2021
Now that RV32 and RV64 have separate implementations, is there still a point in keeping the LOAD/STORE/STRIDE macros?
Feb 28 2021
Feb 26 2021
Feb 25 2021
From some quick testing, it seemed that you could trim this test down a bit.
You're also adding # REQUIRES: asserts but not actually checking for any debug output.
Also, it would be nice to add a comment documenting the logic of this test, unless new CHECKs of the debug output make it more obvious.
Feb 24 2021
Thanks @jrtc27 for the sexti32 review suggestions!
Feb 23 2021
Feb 22 2021
Makes sense to me.
Feb 21 2021