[PATCH] D93878: [RISCV] Don't use tail agnostic policy on instructions where destination is tied to source

Zakk Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 28 20:30:34 PST 2020


khchen added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2151
+  bool TailAgnostic = true;
+  if (MI.isRegTiedToUseOperand(0) && !WritesElement0)
+    TailAgnostic = false;
----------------
If the tail policy is ignored on WritesElement0 instructions, why not set tail policy as `tu`? 
Is it because spec said `If anything, the default should be tail-agnostic/mask-agnostic, so software has to specify when it cares about the non-participating elements`?

If we set anything as `tu` except undefined maskedoff, could it also make vsetvli removal easier.




Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93878/new/

https://reviews.llvm.org/D93878



More information about the llvm-commits mailing list