User Details
- User Since
- May 29 2017, 8:02 AM (266 w, 2 d)
Wed, Jun 29
Rebased to top of trunk.
Removed the extra space.
Wed, Jun 22
Tue, Jun 21
Wed, Jun 15
LGTM
Tue, Jun 14
And, I didn't see the follow up reply from Nemanja.
Okay. I've added the test back.
Removed 32 bit LE test.
Rebased to top of main branch.
Mon, Jun 13
Updated test case to include 32 bit as well as 64 bit tests.
I did check and the issue can been seen for both 32 bit and 64 bit.
Thu, Jun 9
Added test case.
May 31 2022
Rebased the patch to top of main branch.
Updated a couple of typos.
Moved a check earlier in the function.
May 26 2022
May 20 2022
May 19 2022
Rebased to top of main branch.
Apr 29 2022
Rebased patch to top of trunk.
Added the additional #ifndef NDEBUG.
Apr 25 2022
Apr 18 2022
Rebased to top of main branch.
Rebased patch to top of master.
Rebased the patch to top of trunk again.
Apr 14 2022
Added two new flags ZExt32To64 and SExt32To64.
Marked instructions that zero extend or sign extend in the TD files with those
flags.
Fixed a few comments as well.
Tried to extract an if statement to reduce the indentation a little.
Apr 13 2022
Fairly significant changes have been made since I took over this patch and I would prefer to have another set of eyes take a look.
I will request a new review.
Rebased patch to top of trunk.
Apr 11 2022
Need to rebase this patch again. Will request another review once I have done that.
Mar 31 2022
Addressed comment and rebased to top of main.
Addressed comments and rebased patch to top of main.
Mar 24 2022
Rebased the patch to top of main branch.
Mar 23 2022
Mar 21 2022
Mar 18 2022
Mar 15 2022
Rebased patch to top of trunk.
Fixed comment as per review.
Mar 10 2022
Mar 1 2022
Feb 28 2022
Feb 25 2022
Feb 24 2022
Moved the include to PPCInstrInfo.td.
Gentle ping: libc++
Feb 23 2022
Rebased the patch.
Moved the def for LXVKQ lower down.
Feb 22 2022
Feb 18 2022
Updated to use _Pragma pack instead of using the attribute.
Also, made the change only for AIX.
Rebased this patch to the top of main and retested.
Feb 15 2022
MaskRay has fixed the issue with https://reviews.llvm.org/D119787
I checked and this fixes the issue we were seeing.
Thank you for fixing this!
Moved the attribute to the top of the class.
Typo in the comment.
Feb 14 2022
I've updated the packed attribute and added the under-under for it.
Feb 11 2022
Feb 8 2022
Note that I have reviewed this patch along with https://reviews.llvm.org/D118718. The following comment applies to both patches.