[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