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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
