[PATCH] D17293: [MC] AsmLexer: add extensible identifier's character set support.

Valery Pykhtin via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 06:24:29 PST 2016


vpykhtin retitled this revision from "[MC] AsmLexer: 30% speedup on tests, added extensible identifier's character set support." to "[MC] AsmLexer: add extensible identifier's character set support.".
vpykhtin updated the summary for this revision.
vpykhtin updated this revision to Diff 78513.
vpykhtin added a comment.
Herald added subscribers: nhaehnle, wdng.

After a loooong time I would like to reanimate this review requiest.

Previously I incorrectly measured performance impact for this patch and obtained 30% performance gain - this result was incorrect. Current measurement on a large .s file shows no affect on parsing performance.


Repository:
  rL LLVM

https://reviews.llvm.org/D17293

Files:
  include/llvm/MC/MCParser/MCAsmLexer.h
  lib/MC/MCParser/AsmLexer.cpp
  lib/MC/MCParser/MCAsmLexer.cpp
  lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
  test/MC/AMDGPU/hsa.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17293.78513.patch
Type: text/x-patch
Size: 9080 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161118/0dab3fca/attachment.bin>


More information about the llvm-commits mailing list