[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
Tue Jan 15 17:15:35 PST 2019


mgrang marked an inline comment as done.
mgrang added inline comments.


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


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