[PATCH] D113173: [AsmPrinter][ORE] use correct opcode name
Jinsong Ji via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 4 12:02:13 PDT 2021
jsji added inline comments.
================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1418
for (auto &KV : MnemonicVec) {
- auto Name = (Twine("INST_") + KV.first.trim()).str();
+ auto Name = (Twine("INST_") + KV.first.trim().take_until([](char c) {
+ return c == ' ' || c == '\t' || c == '\n' || c == '\v' ||
----------------
Can we just use getToken()?
```
- auto Name = (Twine("INST_") + KV.first.trim()).str();
+ auto Name = (Twine("INST_") + getToken(KV.first).first.trim()).str();
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113173/new/
https://reviews.llvm.org/D113173
More information about the llvm-commits
mailing list