[PATCH] D25927: [cfi] Implement cfi-icall using inline assembly.

[Eric Christopher via llvm-commits]

echristo added a comment.

In https://reviews.llvm.org/D25927#581379, @pcc wrote:

> LGTM if Eric is ok with this.
>
> Eric: in the long term do we want the mangler exposed to Transforms? Ideally Transforms should just be dealing with GlobalValues, correct? If that's the case, it doesn't seem worth threading TLOF through to here.


Long term it's interesting - I think if you're creating named symbols in a Transform we're going to need to plug this through, and TLOF is a better option for doing that than everyone calling various (different) versions of the Mangler/TLOF/TM current API that I'm cleaning up.

-eric


Repository:
  rL LLVM

https://reviews.llvm.org/D25927