[PATCH] D78545: [RISCV] Make CanLowerReturn protected for downstream maintenance

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 11 23:25:41 PDT 2020


jrtc27 added a comment.

I am still very confused why this is necessary. `CanLowerReturn` barely does anything other have a loop to call other functions that actually do useful work, it seems like the wrong place to hook into things. Your "explanation" is merely just "we want this because we want this". Could you please shed some light on how _exactly_ this is useful? As it stands I am none the wiser.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78545





More information about the llvm-commits mailing list