[PATCH] D141759: [WIP][7/N][Clang][RISCV][NFC] Correct the default value for Policy to undisturbed

Yueh-Ting (eop) Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 14 05:22:14 PST 2023


eopXD created this revision.
Herald added subscribers: sunshaoce, VincentWu, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, simoncook, johnrusso, rbar, asb, arichardson.
Herald added a project: All.
eopXD requested review of this revision.
Herald added subscribers: cfe-commits, pcwang-thead, MaskRay.
Herald added a project: clang.

The default value is set to `Omit`, but in fact the value is set under
`updateNamesAndPolicy`, which is set to TUMU when masked and TAMU when
unmasked.

This commit demonstrates so and further remove another dependency of
`Omit.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D141759

Files:
  clang/include/clang/Support/RISCVVIntrinsicUtils.h


Index: clang/include/clang/Support/RISCVVIntrinsicUtils.h
===================================================================
--- clang/include/clang/Support/RISCVVIntrinsicUtils.h
+++ clang/include/clang/Support/RISCVVIntrinsicUtils.h
@@ -99,8 +99,8 @@
     Agnostic,
     Omit, // No policy required.
   };
-  PolicyType TailPolicy = Omit;
-  PolicyType MaskPolicy = Omit;
+  PolicyType TailPolicy = Undisturbed;
+  PolicyType MaskPolicy = Undisturbed;
   Policy() : IsUnspecified(true) {}
   Policy(PolicyType _TailPolicy, PolicyType _MaskPolicy)
       : TailPolicy(_TailPolicy), MaskPolicy(_MaskPolicy) {}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D141759.489233.patch
Type: text/x-patch
Size: 611 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230114/9b7c5759/attachment-0001.bin>


More information about the cfe-commits mailing list