[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:45:10 PDT 2021


nickdesaulniers added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:45
+  }
+  return true;
+}
----------------
samitolvanen wrote:
> nickdesaulniers wrote:
> > Can llvm::any_of or llvm::none_of be used here?
> > llvm/ADT/STLExtras.h
> Maybe, but I don't see how they would make this function any cleaner. Did you have something specific in mind?
Something like?

return any_of(Name, [](const char &C) { return isAlnum(C) || C == '_' || C == '.'; }

or maybe we need !none_of(...)? (not sure if characters of a string can be enumerated this way)


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