[PATCH] D95227: [RISCV] Simplify BP initialisation
Jessica Clarke via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 25 14:22:15 PST 2021
jrtc27 accepted this revision.
jrtc27 added a comment.
This revision is now accepted and ready to land.
In D95227#2521189 <https://reviews.llvm.org/D95227#2521189>, @luismarques wrote:
> This is certainly cleaner, in the sense of the code operating at a higher level of abstraction. I wonder if it's worth the indirection cost.
> I'll let others bikeshed this :-)
Fewer places we have to teach to use a `cmove cbp, csp` dowstream in CHERI-LLVM :) (although we've already paid that cost, but still, diff reducing is good)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95227/new/
https://reviews.llvm.org/D95227
More information about the llvm-commits
mailing list