[PATCH] D56747: [EH] Rename llvm.x86.seh.recoverfp intrinsic to llvm.eh.recoverfp

Mandeep Singh Grang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 1 12:38:56 PST 2019


mgrang marked an inline comment as done.
mgrang added inline comments.
Herald added a project: LLVM.


================
Comment at: llvm/trunk/lib/IR/AutoUpgrade.cpp:547
     }
+    if (Name == "x86.seh.recoverfp") {
+      NewFn = Intrinsic::getDeclaration(F->getParent(), Intrinsic::eh_recoverfp);
----------------
mgrang wrote:
> cuviper wrote:
> > This is under `switch (Name[0]) { case 'a': ` -- so `x86` will never match here.
> Thanks @cuviper. Will fix this.
This is fixed in https://reviews.llvm.org/D57614.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56747





More information about the llvm-commits mailing list