[PATCH] D104058: ThinLTO: Fix inline assembly references to static functions with CFI

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 30 11:08:35 PDT 2021


nickdesaulniers added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:39
+  // Promotion aliases are used only in inline assembly. It's safe to
+  // simply skip unusual names. Matches MCAsmInfo::isAcceptableChar().
+  for (const char &C : Name) {
----------------
What about `MCAsmInfoXCOFF::isAcceptableChar`?  I guess we could be targeting a XCOFF object file format with LTO?


================
Comment at: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:45
+  }
+  return true;
+}
----------------
Can llvm::any_of or llvm::none_of be used here?
llvm/ADT/STLExtras.h


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104058



More information about the cfe-commits mailing list