- User Since
- Nov 9 2018, 8:48 AM (106 w, 5 d)
Thu, Nov 19
Wed, Nov 18
Jun 12 2020
Jun 10 2020
Sorry, I forgot the clang-format.
I added support for the case when exponent is 0.25 in addition to support for double precision cases.
Jun 9 2020
Moving completely the changes from PPCISelLowring.cpp to PPCLowerMASSVEntries.cpp (MASSV pass) to address the reviewer comments.
Jun 4 2020
I think I had done a terrible mistake. When I tested with my c code, I didn't have if (ClVectorLibrary == TargetLibraryInfoImpl::MASSV) on
if (ClVectorLibrary == TargetLibraryInfoImpl::MASSV) setOperationAction(ISD::FPOW, MVT::v4f32, Custom);
and for some reason if I move this check inside the function LowerFPOWMASSV it works good. So I am updating the patch. Thank you for catching it.
Jun 3 2020
Jun 2 2020
Jun 1 2020
Addressing the reviews
May 28 2020
Apr 30 2020
Apr 29 2020
Removing other dependencies to PowerPC machines in the test suits.
Apr 28 2020
Oh, sorry I missed these part in my last update.
Apr 27 2020
Sorry, I was missing your suggestion of commenting above check on the test file.
Reviews are addressed.
Thanks Francesco for the reviews.
The reviews are addressed.
Apr 24 2020
Addressing comments and adding unit tests.
Apr 22 2020
Apr 17 2020
Assert is required for this test.
Apr 16 2020
I am adding a test to my proposed change.
Apr 14 2020
Apr 13 2020
Feb 28 2020
@Whitney @fpetrogalli @pjeeva01:
Thank you guys for those comments. About the test, as @fpetrogalli said test for attribute generation is available llvm/test/Transforms/Util/add-TLI-mappings.ll for new pass manager. And I added the test to see if it is invoked in pipeline here in this patch. As @pjeeva01 said, I need to add a test to see if the vectorized function is called in presence of the vector library option.